Repository navigation
ci: run release admission contracts once per policy job - #1437
Merged
Merged
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
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.
Change
CI policy contracts ran
test-release-once.shtwice on the same checkout: through the selector suite and through the release artifact suite. Remove the selector's duplicate invocation. The release artifact suite retains all release admission and source identity cases. Workflow selection, required gates, production scripts, and test assertions are unchanged. Standalone selector tests now cover scheduling without also running the release admission suite.The previous CI history report ranked only four installation candidates. This is the next independently actionable improvement after reassessment, not a previously measured fifth-place ranking. Large research builds, nextest shards and collector parallelization remain owned by other PRs.
Issue relationship
None
Validation
a84531f51092f9dc52bbee2efac5edcbce8c2778. Full selector suite passed before and after: 87.417 s → 60.259 s locally, a single sequential pair (27.158 s saved; timing varies by host/load).git diff --checkpassed. Diff review confirms only the redundant invocation and its explanatory comment changed.3003131fbba2d03bddc42100e7f858b6a3da959c.Runtime impact and rollback
CI test orchestration only. Revert this commit to restore the duplicate execution. Existing automatic publication selection is unchanged. No production transition is part of this change.