{{ message }}
Conversation
zozo123
requested review from
amoghrajesh,
ashb,
bugraoz93,
gopidesupavan,
jason810496,
jscheffl and
potiuk
as code owners
September 14, 2026 13:34
boring-cyborg
Bot
added
area:dev-tools
backport-to-v3-3-test
Keep selected test commands and supported matrices unchanged. Validate prek hook environments after restore, use a host-aware cache identity and bounded fast compression, and remove duplicate UI/browser archive transfers. Preserve cleanup failure handling and migration environment coverage. Add 133 targeted setup and workflow regression cases. Generated-by: ChatGPT
Use one existing shard per selected Python/backend environment for migration validation instead of provisioning another runner and restoring the image again. Preserve the original test check names and propagate migration failures through an existing check, without requiring branch-rule changes. Keep all test commands, matrix exclusions and the unsupported-Python guard. Generated-by: ChatGPT
Static checks failed on the new test module: ruff removed a redundant blank line before the first-party import, and ruff format normalized the raw string quotes, a comprehension, and an implicit string concatenation. Also materialize the itertools.product argvalues for the cache-refresh-policy parametrization as a list, so pytest does not emit PytestRemovedIn10Warning for a non-Collection iterable. All 133 cases in scripts/tests/ci/test_ci_setup_optimizations.py still pass, including with -W error::DeprecationWarning. Generated-by: ChatGPT Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
This was referenced Sep 15, 2026
Contributor
Author
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
2 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.
Reduce repeated CI setup work while preserving the selected test commands and supported Python/backend environments.
node_modulesarchives. Install only the selected Playwright engine, with support forallretained.Validation: all 133 cases in
scripts/tests/ci/test_ci_setup_optimizations.pypass in the Python 3.10 Breeze environment. These exercise cache identity and recovery, archive permissions and symlinks, browser selection, cleanup failure propagation, and migration environment coverage.The initial upstream run exposed formatting issues in the new tests, now corrected. It also encountered external download failures (Gradle HTTP 504 and a connection reset during Helm schema validation). Fresh upstream CI must pass before merge. No end-to-end speedup is claimed without comparable completed runs.
Was generative AI tooling used to co-author this PR?
Generated-by: ChatGPT and Codex (GPT-6) following the guidelines