Skip to content

ci: cancel superseded runs of bridge-gate, governance and hypatia-scan - #1045

Merged
hyperpolymath merged 2 commits into
mainfrom
claude/workflow-concurrency
Nov 10, 2026
Merged

hyperpolymath merged 2 commits into
mainfrom
claude/workflow-concurrency

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Why

On 2026-11-09 between 16:12 and 16:23 UTC, 7 PR branches were pushed within about 10 minutes. Their runs waited about two hours for runners: created at 16:12, jobs started around 18:05. In tests.yml, Integration Coverage never got a runner, so Integration Status reported it as abandoned and failed. Example: run 42652388070, where Build Test Images and Integration Tests passed and Integration Coverage has no job record.

9 of the 37 workflows have no concurrency group, so a newer push does not free the runner an older run holds.

What

The block that tests.yml already uses is added to the 3 workflows that run on every PR push and only read and report:

concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true
  • bridge-gate.yml, governance.yml, hypatia-scan.yml.

Left alone on purpose:

  • label-triage.yml: issue events share one group, so one issue would cancel another's triage.
  • mirror.yml, labels.yml, zz-one-off-lock-regen.yml: a cancel mid-run could leave a mirror, the label set or a pushed commit half done.
  • rsr-oracle.yml: a reusable workflow; its caller owns concurrency.
  • scorecard.yml: schedule and dispatch only, so it never holds a PR runner.

Testing

  • yq '.concurrency' parses on all three files. No uses: line changed, so actions.lock is unaffected.
  • No upload, push or merge step in the three; a cancelled run loses only a superseded result.

Generated by Claude Code

These three run on every PR push and had no concurrency group, so a newer
push left the older run holding a runner. With many PR pushes at once (the
2026-11-09 16:12-16:23 burst) jobs waited about two hours, and tests.yml's
Integration Coverage was abandoned unrun. The block is the one tests.yml
already uses.

Not changed, on purpose: label-triage.yml (issue events share one group, so
one issue would cancel another's triage); mirror.yml, labels.yml and
zz-one-off-lock-regen.yml (a cancel mid-run could leave a mirror, the label
set or a pushed commit half done); rsr-oracle.yml (reusable; the caller
owns concurrency); scorecard.yml (schedule and dispatch only).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136P7ZnmBahbhfkFEP6ark4
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 4 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 125003cb-e1d6-4013-af0c-92723dd56f4b
Commits

Reviewing files that changed from the base of the PR and between 9c8c77d and 23e34ad.

Files selected for processing: 3

  • .github/workflows/bridge-gate.yml
  • .github/workflows/governance.yml
  • .github/workflows/hypatia-scan.yml
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@sonarqubecloud

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath marked this pull request as ready for review November 10, 2026 16:39
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-10T16:41:26.654219Z 23e34ad Draft marked ready
🔒 Security Review ❌ Failed 2026-11-10T16:39:13.504036Z
We couldn’t complete this review. Comment @codex security review to try again.
23e34ad Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

This security review couldn’t start because the paying account has exhausted its Codex allowance and has insufficient available credits. The payer can check their balance and add credits in the Codex usage dashboard, then try again. If you do not manage the paying account, contact this repository's admins.

@hyperpolymath
hyperpolymath enabled auto-merge (squash) November 10, 2026 16:39
@hyperpolymath
hyperpolymath merged commit 4209a52 into main Nov 10, 2026
90 checks passed
@hyperpolymath
hyperpolymath deleted the claude/workflow-concurrency branch November 10, 2026 16:45
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.

3 participants