[18.0-fr7] Drop implied main/master branch matchers pragma - #1219
amartyasinha wants to merge 1 commit into
Conversation
The pragma section declaring `implied-branches: [main, master]` was inherited from main when this stable branch was cut (added on main in 3501282 to alias main/master, where the repo default is main but jobs run against master). On a release branch that alias is wrong: it makes this branch's job definitions (e.g. nova-operator-kuttl, which depends on openstack-meta-content-provider) also match main and master changes. Because of that leak, any main change that moves nova-operator-kuttl off the meta content provider fails to freeze its job graph: Unable to freeze job graph: Job nova-operator-kuttl depends on openstack-meta-content-provider which was not run. Removing the pragma lets Zuul's default per-branch implied matchers scope this branch's config to 18.0-fr7 only (matching how watcher-operator's stable branches, which carry no pragma, behave). No functional change to this branch's own CI. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Amartya Sinha <amsinha@redhat.com>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amartyasinha The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
Build failed (check pipeline). Post ✔️ openstack-meta-content-provider SUCCESS in 3h 13m 46s |
1 participant
The pragma section declaring
implied-branches: [main, master]was inherited from main when this stable branch was cut (added on main in 3501282 to alias main/master, where the repo default is main but jobs run against master). On a release branch that alias is wrong: it makes this branch's job definitions (e.g. nova-operator-kuttl, which depends on openstack-meta-content-provider) also match main and master changes.Because of that leak, any main change that moves nova-operator-kuttl off the meta content provider fails to freeze its job graph:
Unable to freeze job graph: Job nova-operator-kuttl depends on
openstack-meta-content-provider which was not run.
Removing the pragma lets Zuul's default per-branch implied matchers scope this branch's config to 18.0-fr7 only (matching how watcher-operator's stable branches, which carry no pragma, behave). No functional change to this branch's own CI.