Skip to content

[ci] Keep skipped label-gated runs from masking live GPU CI - #2110

Merged
erictang000 merged 1 commit into
NovaSky-AI:mainfrom
erictang000:ci/distinct-skipped-check-names
Sep 1, 2026
Merged

[ci] Keep skipped label-gated runs from masking live GPU CI#2110
erictang000 merged 1 commit into
NovaSky-AI:mainfrom
erictang000:ci/distinct-skipped-check-names

Conversation

@erictang000

Copy link
Copy Markdown
Collaborator

A labeled event starts a run of every workflow triggered on labeled, whatever the label was -- applying three labels to #1898 produced 15 runs, of which 3 did work. The other 12 skip, which is correct, but they published their skip under the same check name as the real run. GitHub keeps only the newest check of a given name, so whether a live suite showed as running came down to whether its own run happened to be created last. Two of the three running suites reported "skipped" on the PR.

Name the job by whether it is going to run, so a skip lands on its own check context and can only ever supersede another skip. The running branch is the job id -- the existing context name -- so any branch protection pointing at it is unaffected.

A `labeled` event starts a run of every workflow triggered on `labeled`,
whatever the label was -- applying three labels to NovaSky-AI#1898 produced 15 runs,
of which 3 did work. The other 12 skip, which is correct, but they published
their skip under the same check name as the real run. GitHub keeps only the
newest check of a given name, so whether a live suite showed as running came
down to whether its own run happened to be created last. Two of the three
running suites reported "skipped" on the PR.

Name the job by whether it is going to run, so a skip lands on its own check
context and can only ever supersede another skip. The running branch is the
job id -- the existing context name -- so any branch protection pointing at
it is unaffected.
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@erictang000
erictang000 merged commit 5424e3d into NovaSky-AI:main Sep 1, 2026
1 of 2 checks passed
@erictang000
erictang000 deleted the ci/distinct-skipped-check-names branch September 1, 2026 00:10
erictang000 added a commit that referenced this pull request Sep 1, 2026
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.

1 participant