Skip to content

log buildkitd worker readiness wait duration in setup step#114

Open
ajwerner wants to merge 1 commit into
mainfrom
devin/1783621186-log-ready-duration
Open

log buildkitd worker readiness wait duration in setup step#114
ajwerner wants to merge 1 commit into
mainfrom
devin/1783621186-log-ready-duration

Conversation

@ajwerner

@ajwerner ajwerner commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

The time spent waiting for buildkitd workers to become ready was only reported as a backend metric (BPA_BUILDKITD_READY_DURATION_MS), invisible in the job log. This adds the elapsed wait to the existing "Found N workers" log line in startAndConfigureBuildkitd, so users can see how long the daemon took to come up directly in the setup step output:

Found 1 workers, required 1 (buildkitd workers ready after 4321ms)

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled. (Staging)


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@ajwerner ajwerner self-assigned this Jul 9, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@ajwerner ajwerner marked this pull request as ready for review July 9, 2026 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant