[CI] Reserve publisher integration for policy changes - #1268
Open
brandonpayton wants to merge 2 commits into
Open
[CI] Reserve publisher integration for policy changes#1268brandonpayton wants to merge 2 commits into
brandonpayton wants to merge 2 commits into
Conversation
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.
Why
Ordinary package and runtime pull requests currently pay about 17 minutes to
exercise the complete privileged Homebrew publication lifecycle. When those
changes do not touch publisher policy, that suite does not test the code that
changed and delays the next useful staging result.
This is especially costly while ABI 43 is iterating through candidate bottle
waves. The publisher suite remains valuable when its own policy changes, but it
should not be a tax on every unrelated iteration.
What changed
install, and complete publisher suite on the existing fail-closed
package_publish_flow_changedscope.mutations.
No Formula, package, ABI, request, candidate, bottle, or publication digest is
changed. This PR is separate from the immutable ABI 43 request.
Validation
actionlint -ignore SC1007 .github/workflows/staging-build.yml: pass.git diff --check: pass.Unfiltered actionlint still reports the pre-existing
SC1007warning atstaging-build.yml:1266; the same warning reproduces againstorigin/main.Because this PR changes publisher-flow policy, its hosted staging preflight is
expected to run the complete publisher suite once. That hosted result is the
integration gate before merge.