ci: Change image-warming pull to happen in parallel - #5849
Draft
backspace wants to merge 2 commits into
Draft
Conversation
The five mirrored images were pulled one after another, so the step cost the sum of five pulls — a median 29s on every host shard, which at 20 shards is ten minutes of machine time per run for work the daemon can do concurrently. Pull them together and wait. Each pull's output is captured and replayed in a fixed order afterwards: interleaved progress from five images is unreadable exactly when someone is trying to see which mirror failed. `warm` already downgrades a miss to a warning and falls back to Docker Hub, so no failure semantics change here. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
Host Test Results 1 files ±0 1 suites ±0 1h 56m 2s ⏱️ - 1m 51s Results for commit 3eef91b. ± Comparison against earlier commit 4533346. Realm Server Test Results 1 files ±0 1 suites ±0 16m 17s ⏱️ +44s Results for commit 3eef91b. ± Comparison against earlier commit 4533346. |
The first run showed the warm step dropping 29s to 13s while the readiness wait grew 1s to 38s. One sample cannot separate that from machine-speed drift, and the run's shards were slower across the board. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
There’s no reason to do this serially.