Skip to content

Stop concurrent sample releases from failing on push races - #8

Closed
brionmario wants to merge 1 commit into
thunder-id:mainfrom
brionmario:fix/release-push-race
Closed

brionmario wants to merge 1 commit into
thunder-id:mainfrom
brionmario:fix/release-push-race

Conversation

@brionmario

Copy link
Copy Markdown
Member

Purpose

The first release of the quickstarts (run 39429836038) failed for quickstarts/express, quickstarts/vue and quickstarts/flutter. All ten selected samples ran in parallel, and each pushes its version-bump commit to main. The commit step retried five times with a fixed 3, 6, 9 and 12 second backoff, so jobs that lost the race kept colliding in step until they gave up:

! [remote rejected] HEAD -> main (cannot lock ref 'refs/heads/main': is at 2eefcda... but expected 70047f8...)

The three failed jobs stopped before tagging, so no tag or release was created for them and nothing needs cleaning up.

Approach

  • Run at most four releases at a time (max-parallel: 4).
  • Retry the push up to ten times, waiting attempt + random(0-9) seconds between attempts so the jobs fall out of step.

Related Issues

Related PRs

Checklist

  • Followed the contribution guidelines.
  • Manual test round performed and verified.
  • Documentation provided. (Add links if there are any)
  • Tests provided. (Add links if there are any)
    • Unit Tests
    • Integration Tests
  • Breaking changes. (Fill if applicable)
    • Breaking changes section filled.
    • breaking change label added.

Security checks

  • Followed secure coding standards.
  • Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.

Releasing many samples at once ran every matrix job's version-bump push against main together, and the fixed 3 to 12 second backoff gave up after five attempts, failing the Express, Vue and Flutter quickstart releases. Run at most four releases at a time and retry up to ten times with a randomized wait.

Refs thunder-id/thunderid#5904

Signed-off-by: Brion <info@brionmario.com>
@brionmario

Copy link
Copy Markdown
Member Author

Combined into #11 together with the other sample changes, one commit each.

@brionmario brionmario closed this Oct 21, 2026
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