ci: register build accelerator routing - #1872
Merged
Merged
Conversation
Contributor
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe pull request updates Windows ownership patterns and refreshes container image digests and toolchain epochs for four Linux runtime rows. ChangesWindows ownership routing
Linux runtime metadata
Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Feature Suggested reviewers: ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
i386
pushed a commit
that referenced
this pull request
Sep 15, 2026
Move the promoted runner catalog and every matching workflow binding into the prerequisite that provisions sccache for protected Quality jobs. This repairs the transitional planner/catalog drift left after #1872 and keeps the release and runtime-seed contracts aligned with the promoted cohort. Validation: just ci-validate components pass, including 1,474 tests with 9 skipped; runner identity reports 9 images, 35 roles, and 36 bindings.
i386
added a commit
that referenced
this pull request
Sep 15, 2026
# Problem PR #1872 moved the protected planner's four Linux runtime rows to the promoted runner cohort, while `ci/runner-images.json` and the remaining workflow bindings still described the previous cohort. That transitional split makes repository identity checks fail on `main`. Separately, the protected Quality workflow did not provision sccache before Cargo invocations, which blocks PR #1870 once the repository-level `rustc-wrapper` becomes mandatory. # Result This prerequisite completes the promoted-image transition before #1870: - admits the promoted nine-image cohort with its retained promotion receipt; - updates all 36 registered workflow bindings and their toolchain epochs; - gives crates.io publish and resume jobs the registered CPU container and local sccache policy; - rotates compiler-seed identity for the new CPU image and prevents branch cache shadowing; - provisions sccache in both protected Quality jobs; - updates the matching catalog, release, cache, and artifact contracts. The linker defaults and Windows short-path build behavior remain in #1870. After this lands, #1870 can rebase onto a consistent protected planner/catalog base. # Validation On exact head `b453d87f289ba71631a33e42f67162dcae5a4f01`: - `just ci-validate`: passed, including 1,474 tests with 9 skipped; - runner image identity: 9 images, 35 roles, 36 workflow bindings, 4 runtime rows, 6 seed consumers; - actionlint and `git diff --check`: passed; - release targets, crate-list consistency, console-print policy, and publish-chain consistency: passed. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated build, test, and release workflows to newer verified runner images. * Refreshed cache keys and expanded caching configuration across CI and release workflows. * Added provenance and receipt details for runner images. * Added dedicated release publishing environments with safer cache settings and checkout validation. * Restricted runtime seed canary execution to the main branch. * **Tests** * Expanded validation for runner image provenance, cache identity, release workflows, and canary behavior. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: scama <a1860575018c4680d5669dd7bc3bd356b478bccb8d42e194df46304a5e25f49a@meshllm.communities.buzz.xyz>
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.
The protected PR planner requires
ci/ownership.ymlandci/slices.ymlto match the default branch byte for byte. This prerequisite publishes the routing data needed by #1870 so that the feature branch can be rebased and validated.This is the repository's documented sequenced-maintainer-merge path for catalog evolution. The catalog PR's own Plan checks are expected to reject the source/default mismatch until it lands.
Validation: 53 planner/lane contract tests passed;
just ci-crate-listspassed.Summary by CodeRabbit