Skip to content

fix(ci): serialize release-please runs on consecutive main pushes - #173

Merged
sebastiancorrea81 merged 1 commit into
mainfrom
fix/release-please-concurrency
Aug 11, 2026
Merged

fix(ci): serialize release-please runs on consecutive main pushes#173
sebastiancorrea81 merged 1 commit into
mainfrom
fix/release-please-concurrency

Conversation

@sebastiancorrea81

Copy link
Copy Markdown

Problem

Two pushes to main landing close together let two release-please runs overlap: one merges/creates the release PR while another run, already mid-flight with stale state, tries to update the PR that just got merged and fails:

⚠ updated code for 170, but update requested for 169
##[error] ... 422 the pull request cannot be reopened

Observed live on 2026-08-07 (PR #169/#170, ~90s window). Same signature as nullplatform/tofu-modules#485's underlying issue.

Fix

Add the same top-level concurrency group that nullplatform/tofu-modules' release.yml already uses, so overlapping pushes to main queue instead of racing. cancel-in-progress: false because cancelling a run mid-flight could interrupt release-please while it's mutating GitHub state (tags, releases, labels).

Note: this doesn't fully eliminate the class of race seen in tofu-modules#485 (that one happened even with this same concurrency block, from two runs executing back-to-back rather than in parallel) — but it does close the specific parallel-execution case seen here, which is a wider and easier-to-hit window.

🤖 Generated with Claude Code

Two pushes to main landing within the ~45s auto-merge window let two
release-please runs overlap: one merges/creates the release PR while
another, already mid-flight with stale state, tries to update the PR that
just got merged and fails with "the pull request cannot be reopened".
Observed live on 2026-08-07 (PR #169/#170) and previously on
nullplatform/tofu-modules#485 with the same signature.

Adds the same top-level concurrency group tofu-modules' release.yml
already uses, so overlapping pushes queue instead of racing.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@sebastiancorrea81
sebastiancorrea81 merged commit ae661d8 into main Aug 11, 2026
4 checks passed
@sebastiancorrea81
sebastiancorrea81 deleted the fix/release-please-concurrency branch August 11, 2026 14: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.

2 participants