ci: single required aggregator (CI Required) over the 10 required contexts#904
Draft
frstrtr wants to merge 1 commit into
Draft
ci: single required aggregator (CI Required) over the 10 required contexts#904frstrtr wants to merge 1 commit into
frstrtr wants to merge 1 commit into
Conversation
…context workflows Introduce a single reusable-workflow orchestrator (CI Required) that calls coin-matrix + the 5 gate workflows and ends in an always()-run aggregate job. The aggregate fails on any failed/cancelled lane and tolerates skipped, so branch protection can require one context (CI Required / aggregate) in place of the 10 individual contexts. This lets path-filtering skip untouched lanes on a PR without blocking the merge. workflow_call: is added additively to the 6 workflows; their existing push and pull_request triggers and behaviour are unchanged.
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.
Reusable-workflow orchestrator so branch protection can require ONE context instead of 10, unblocking path-filter skips.
What
ci-required.yml(workflow name:CI Required) fires onpull_requestto master. It calls the 6 workflows carrying the 10 required contexts (coin-matrix + DASH G1/G3a + DGB Phase-B + BCH G3a/G3b) viaworkflow_callwithsecrets: inherit, then runs anaggregatejob (needs: [all],if: always()).aggregatefails on any lane that is failure/cancelled and toleratesskipped— so a PR that path-filters an untouched lane still merges.workflow_call:added additively to the 6 workflows; their existing push/pull_request triggers and behaviour are unchanged.Intended required context (post-merge, operator re-points branch protection)
CI Required / aggregate— replaces the 10 individual contexts.Transition note
Until branch protection is re-pointed and the legacy direct triggers retired, the 6 lanes run both directly and under the aggregator on a PR. No concurrency collision: only coin-matrix has a concurrency group and it keys on
github.workflow, which differs between the direct run and the reusable call.Draft — held for operator branch-protection re-point after the single settle run.