Skip to content

t3532: fix concurrency group in publish-packages and postflight workflows#4710

Merged
marcusquinn merged 1 commit intomainfrom
fix/quality-debt-3532-workflow-concurrency-group
Mar 14, 2026
Merged

t3532: fix concurrency group in publish-packages and postflight workflows#4710
marcusquinn merged 1 commit intomainfrom
fix/quality-debt-3532-workflow-concurrency-group

Conversation

@alex-solovyev
Copy link
Collaborator

Summary

  • Incorporates the version/tag workflow_dispatch input into the concurrency group key so distinct manual runs no longer cancel each other
  • Falls back to github.sha for release-triggered runs (where no input is provided), preserving existing deduplication behaviour for automated triggers

Changes

File Before After
publish-packages.yml ${{ github.workflow }}-${{ github.ref }} `${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.version
postflight.yml ${{ github.workflow }}-${{ github.ref }} `${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.tag

Why

Two workflow_dispatch runs targeting different versions (e.g., re-publishing v2.51.0 while v2.52.0 is in flight) shared the same concurrency group and the second would cancel the first. The fix makes each distinct version/tag input produce a unique group key.

Closes #3532

…cancellation of distinct manual runs

Closes #3532

publish-packages.yml: group now includes ${{ github.event.inputs.version || github.sha }}
postflight.yml: group now includes ${{ github.event.inputs.tag || github.sha }}

Without this, two manual workflow_dispatch runs with different version/tag inputs
shared the same concurrency group and the second would cancel the first.
@gemini-code-assist
Copy link

Note

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 14, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • no-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 99e0a9f4-86c2-45d7-9506-b4415217a103

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/quality-debt-3532-workflow-concurrency-group
📝 Coding Plan
  • Generate coding plan for human review comments

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added the bug Auto-created from TODO.md tag label Mar 14, 2026
@github-actions
Copy link

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 347 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Sat Mar 14 08:09:24 UTC 2026: Code review monitoring started
Sat Mar 14 08:09:24 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 347

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 347
  • VULNERABILITIES: 0

Generated on: Sat Mar 14 08:09:27 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link

@marcusquinn marcusquinn merged commit ace6d6c into main Mar 14, 2026
19 checks passed
@marcusquinn marcusquinn deleted the fix/quality-debt-3532-workflow-concurrency-group branch March 14, 2026 08:16
alex-solovyev added a commit that referenced this pull request Mar 14, 2026
…cancellation of distinct manual runs (#4710)

Closes #3532

publish-packages.yml: group now includes ${{ github.event.inputs.version || github.sha }}
postflight.yml: group now includes ${{ github.event.inputs.tag || github.sha }}

Without this, two manual workflow_dispatch runs with different version/tag inputs
shared the same concurrency group and the second would cancel the first.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Auto-created from TODO.md tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

quality-debt: .github/workflows/publish-packages.yml — PR #290 review feedback (medium)

2 participants