Skip to content

ci: stop the merge gatekeeper waiting on Graphite's mergeability check - #14959

Open
asaf-sw wants to merge 1 commit into
mainfrom
asaf/ci-09-gatekeeper-graphite-check
Open

ci: stop the merge gatekeeper waiting on Graphite's mergeability check#14959
asaf-sw wants to merge 1 commit into
mainfrom
asaf/ci-09-gatekeeper-graphite-check

Conversation

@asaf-sw

@asaf-sw asaf-sw commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Stops the merge gatekeeper timing out on every stacked PR. The cause is Graphite / mergeability_check, which never completes while a PR's base is another open PR, so the gate waits out its full hour and dies reporting zero failed jobs.


Detailed Summary for AI Bots

Problem

Failed job count:      0
Error: context deadline exceeded

It self-resolves as the stack merges bottom-up, which is worse than a hard failure: it trains reviewers to ignore a red gate.

Root cause

Diffing the gate's own "All jobs" and "Completed jobs" groups across all 120 polls of run 31624863250 leaves exactly one job that never completes:

last All jobs: 16   last Completed: 15
NEVER COMPLETED -> ['Graphite / mergeability_check']

The correlation with base branch is exact: #14938 and #14946 (base main) pass; #14944, #14947 and #14948 (stacked) time out. Skipped checks were a red herring, since #14944 and #14947 had zero skips.

Change

Add Graphite / mergeability_check to the gate's ignored list on the pull_request step. It is a stack-state indicator, not a CI signal about the code, and the list already ignores code-review/reviewable for the same reason. The merge_group step is untouched, since Graphite does not post this check against a merge queue ref.

Name matching is substring-based, as the single build_docker_images entry covering all three matrix variants shows.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants