Merged
Conversation
matthewborden
force-pushed
the
preflight-show-summary-api
branch
from
fde95fe to
bd7cf33
Compare
Changed the way that the CLI implements rate limits and the back off. We've removed the dependency on `roko`, which shouldn't be required with the availability of the `RateLimit-` headers that we can get from client calls.
Amp-Thread-ID: https://ampcode.com/threads/T-019d7647-9ca8-77ad-8ca6-af7af5562342 Co-authored-by: Amp <[email protected]>
Amp-Thread-ID: https://ampcode.com/threads/T-019d7652-4ebb-721f-921a-f92bcb53ed55 Co-authored-by: Amp <[email protected]>
Amp-Thread-ID: https://ampcode.com/threads/T-019d7710-3c17-74a6-a2d4-951406f8e5e3 Co-authored-by: Amp <[email protected]>
Builds can have multiple runs for the same suite.
matthewborden
force-pushed
the
preflight-show-summary-api
branch
from
bd7cf33 to
4b35824
Compare
wolfeidau
reviewed
Apr 21, 2026
Member
wolfeidau
left a comment
wolfeidau
left a comment
There was a problem hiding this comment.
couple of low priority things.
matthewborden
force-pushed
the
preflight-show-summary-api
branch
from
03e9e77 to
a8845ce
Compare
This doesn't fit on one line.
mcncl
approved these changes
Apr 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This branch adds a server-aggregated test summary to bk preflight so the final output is more useful once a build completes.
The summary shows per-run results for the build, where each run maps to a test suite, and includes aggregated passed / failed / skipped counts. Failed tests are filtered to group-only-failed, so the summary excludes tests where at least one execution wasn't failed.
A new --await-test-results[=] flag lets bk preflight wait for Test Engine results to be processed before fetching the final summary. When enabled without a value, it waits for the default duration
before loading the aggregated results.
In interactive output, the TUI shows the top 5 failed test results alongside the per-run summary. In JSON output, the summary includes up to the top 100 failed test results.
This change depends on the server-side preflight summary implementation in buildkite/buildkite#29097 (https://github.com/buildkite/buildkite/pull/29097).