Skip to content

Add concurrency group to PR Tooling Safety Check workflow#19841

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/aw-pr-tooling-safety-check-failure
Draft

Add concurrency group to PR Tooling Safety Check workflow#19841
Copilot wants to merge 2 commits into
mainfrom
copilot/aw-pr-tooling-safety-check-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 28, 2026

The push-repo-memory job failed with remote: fatal error in commit_refs when pushing to the safety/scanned-PRs branch. This is a server-side race condition triggered when multiple workflow runs attempt to push to the same branch concurrently.

  • Add explicit concurrency group with cancel-in-progress: false to ensure at most one run executes at a time (queued runs still complete, no scans lost)
  • Recompile lock file
concurrency:
  group: labelops-pr-security-scan
  cancel-in-progress: false

Follows the same pattern already used by labelops-pr-maintenance.md.

…t repo-memory pushes

Co-authored-by: T-Gro <46543583+T-Gro@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix PR Tooling Safety Check failure Add concurrency group to PR Tooling Safety Check workflow May 28, 2026
Copilot finished work on behalf of T-Gro May 28, 2026 09:28
Copilot AI requested a review from T-Gro May 28, 2026 09:28
Copy link
Copy Markdown
Member

@T-Gro T-Gro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The fix correctly serializes workflow runs to prevent the race condition on push-repo-memory pushes to safety/scanned-PRs.

  • Static concurrency group name (labelops-pr-security-scan) is unambiguous — no risk of expression-resolution issues.
  • cancel-in-progress: false ensures queued scans still complete (no lost work).
  • Lock file changes are expected recompilation artifacts from the source change.
  • Follows the same pattern as labelops-pr-maintenance.

@T-Gro T-Gro added the AI-reviewed PR reviewed by AI review council label May 28, 2026
@T-Gro T-Gro self-requested a review May 28, 2026 15:01
@github-actions
Copy link
Copy Markdown
Contributor

✅ No release notes required

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

Labels

AI-reviewed PR reviewed by AI review council

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

[aw] PR Tooling Safety Check failed

2 participants