Skip to content

VED-1101 added concurrency guard to CD pipeline#1271

Merged
avshetty1980 merged 2 commits intomasterfrom
VED-1101-prevent-concurrent-PR-deployment
Mar 6, 2026
Merged

VED-1101 added concurrency guard to CD pipeline#1271
avshetty1980 merged 2 commits intomasterfrom
VED-1101-prevent-concurrent-PR-deployment

Conversation

@avshetty1980
Copy link
Copy Markdown
Contributor

Summary

Adds a concurrency block to continuous-deployment pipeline to prevent simultaneous merges (on master) from causing Terraform state lock collisions and therefore fails.

Used a hardcoded group value to provide self-documenting and decoupled from display metadata.
${{ github.workflow }}-${{ github.ref }} for the CD workflow would just translate to "Continuous Deployment Pipeline/refs/heads/master" - does not change dynamically.
cancel-in-progress: false enables queueing not cancelling the previous run.

  • Routine Change

Reviews Required

  • Dev
  • Test
  • Tech Author
  • Product Owner

@dlzhry2nhs dlzhry2nhs self-requested a review March 5, 2026 16:43
dlzhry2nhs
dlzhry2nhs previously approved these changes Mar 5, 2026
edhall-nhs
edhall-nhs previously approved these changes Mar 5, 2026
Comment thread .github/workflows/continuous-deployment.yml Outdated
Thomas-Boyle
Thomas-Boyle previously approved these changes Mar 6, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 6, 2026

@avshetty1980 avshetty1980 temporarily deployed to internal-dev-sandbox March 6, 2026 09:31 — with GitHub Actions Inactive
@avshetty1980 avshetty1980 temporarily deployed to internal-dev-sandbox March 6, 2026 09:31 — with GitHub Actions Inactive
@avshetty1980 avshetty1980 temporarily deployed to internal-dev-sandbox March 6, 2026 09:32 — with GitHub Actions Inactive
@avshetty1980 avshetty1980 merged commit 7309274 into master Mar 6, 2026
40 of 43 checks passed
@avshetty1980 avshetty1980 deleted the VED-1101-prevent-concurrent-PR-deployment branch March 6, 2026 10:42
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