Skip to content

Commit d79b3cd

Browse files
committed
[CI] Give push events a unique concurrency group
The concurrency group keys on github.event.pull_request.number, which is set only for pull_request events. On a push to master that expression is empty, so every master push collapses into one group and, with cancel-in-progress, each new push cancels the previous run mid-flight. The self-hosted coverage row (wake-on-lan, then a CUDA + Enzyme + sanitizer build) is by far the slowest job, so on master it is almost always cancelled before the codecov upload step runs. Master never gets a base coverage report, and every PR's codecov/project status -- which uses `target: auto` and needs the base coverage -- stalls at "Expected -- Waiting" indefinitely. codecov/patch is unaffected: it needs only the PR's own report. Fall back to github.sha when there is no PR number, so each pushed commit gets its own group and push runs stop cancelling each other.
1 parent 53ebd00 commit d79b3cd

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ on:
88
- master
99

1010
concurrency:
11-
group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
11+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}
1212
cancel-in-progress: true
1313

1414
jobs:

0 commit comments

Comments
 (0)