Skip to content

Prevent duplicate workflow failure issues - #351

Merged
trask merged 1 commit into
mainfrom
trask-concurrency-only-workflow-fix
Sep 4, 2026
Merged

Prevent duplicate workflow failure issues#351
trask merged 1 commit into
mainfrom
trask-concurrency-only-workflow-fix

Conversation

@trask

@trask trask commented Sep 4, 2026

Copy link
Copy Markdown
Member

Serializes workflow-failure issue updates by workflow name so concurrent runs cannot both observe that no tracking issue is open and create duplicates.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@trask
trask marked this pull request as ready for review September 4, 2026 00:03
@trask
trask requested a review from a team as a code owner September 4, 2026 00:03
@trask
trask requested review from mx-psi and a balanced review from Copilot September 4, 2026 00:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The reviewed change safely serializes issue updates, with no unresolved issues.

Pull request overview

Adds workflow-level concurrency to prevent duplicate failure-tracking issues.

Changes:

  • Serializes updates by calling workflow name.
  • Queues later updates without canceling active runs.
File summaries
File Description
.github/workflows/workflow-failure-issue.yml Adds a workflow-specific concurrency group and cancellation policy.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Sep 4, 2026

Copy link
Copy Markdown

Pull request dashboard status

Waiting on maintainers · refreshed 2026-09-04 08:25 UTC

Merge when ready.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused concurrency change has no unresolved issues.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@trask
trask merged commit bb9d47b into main Sep 4, 2026
8 checks passed
@trask
trask deleted the trask-concurrency-only-workflow-fix branch September 4, 2026 17:00
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.

4 participants