Skip to content

fix(release-please): tolerate label-application race#355

Closed
JacobPEvans-personal wants to merge 2 commits into
mainfrom
fix/release-please-label-race
Closed

fix(release-please): tolerate label-application race#355
JacobPEvans-personal wants to merge 2 commits into
mainfrom
fix/release-please-label-race

Conversation

@JacobPEvans-personal
Copy link
Copy Markdown
Owner

Summary

  • Adds continue-on-error: true to the label-application step to tolerate a race condition where the PR is not yet visible when the label workflow fires
  • Re-fails on genuine errors when no PR is found (distinguishes the race from a real failure)

Test plan

  • Release-please runs without spurious failures when PR is created and labelled concurrently
  • Genuine errors still surface (not silently swallowed)

…-error

The googleapis/release-please-action@v5 step opens the release PR
successfully but then immediately tries to apply `autorelease: pending`
via GraphQL. The PR's global node ID hasn't propagated yet at that point,
so GitHub returns "Could not resolve to a node" — failing the job even
though the PR was created correctly.

Adding `continue-on-error: true` lets the downstream `Find release PR
number` step discover the PR via REST and continue normally. The missing
label is cosmetic; release-please re-applies it on the next run.

(claude)
continue-on-error masks all release-please failures, not just the transient
label-application race. Add a guard step that re-fails the job when the
release step errored AND the downstream REST lookup finds no release PR —
distinguishing a genuine auth/config/API failure from the harmless race.

(claude)
@gemini-code-assist
Copy link
Copy Markdown

Note

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

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.

1 participant