Skip to content

Commit fa22ccd

Browse files
authored
fix(ci): publish merge queue E2E statuses (#4370)
Signed-off-by: Evan Lezar <elezar@nvidia.com>
1 parent 82bc756 commit fa22ccd

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

‎.github/workflows/required-ci-gates.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -280,5 +280,8 @@ jobs:
280280
if [ "$CONTEXT_KIND" != "merge_group" ]; then
281281
evaluate_workflow "OpenShell / E2E" "branch-e2e.yml" "Branch E2E Checks" "test:e2e" "Core E2E result"
282282
evaluate_workflow "OpenShell / GPU E2E" "branch-e2e.yml" "Branch E2E Checks" "test:e2e-gpu" "GPU E2E result"
283+
else
284+
post_status "OpenShell / E2E" success "Not applicable: E2E runs only on PRs" "$STATUS_TARGET_URL"
285+
post_status "OpenShell / GPU E2E" success "Not applicable: GPU E2E runs only on PRs" "$STATUS_TARGET_URL"
283286
fi
284287
evaluate_workflow "OpenShell / Helm Lint" "helm-lint.yml" "Helm Lint"

‎CI.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -487,7 +487,7 @@ its own stable result status.
487487
Merge-group runs use the `merge_group` event. The event is distinct from `pull_request` and `push`, and GitHub will not report required checks for queued PRs unless the workflows include it. In this repository:
488488

489489
- `Branch Checks` runs the standard non-E2E gates on the merge-group SHA.
490-
- `Branch E2E Checks` does not run for merge groups. E2E suites remain opt-in on PRs.
490+
- `Branch E2E Checks` does not run for merge groups. E2E suites remain opt-in on PRs. `Required CI Gates` publishes successful not-applicable statuses for both E2E contexts on merge-group events and subsequent merge-group workflow completions.
491491
- `Helm Lint` runs for merge groups without the PR diff optimization, because the merge-group branch is the final integration state.
492492
- `Trivy Changes` compares the merge-group configuration with its base and rejects new High or Critical findings.
493493
- `Required CI Gates` posts the same `OpenShell / ...` statuses to the merge-group SHA and does not require a `pull-request/<N>` mirror for merge-group events.

0 commit comments

Comments
 (0)