Skip to content

fix: prevent duplicate GitHub releases in publish workflow#4489

Merged
lwshang merged 2 commits intomasterfrom
fix/publish-workflow-duplicate-releases
Mar 4, 2026
Merged

fix: prevent duplicate GitHub releases in publish workflow#4489
lwshang merged 2 commits intomasterfrom
fix/publish-workflow-duplicate-releases

Conversation

@lwshang
Copy link
Copy Markdown
Contributor

@lwshang lwshang commented Mar 3, 2026

Summary

  • The publish job used a matrix strategy that ran 4 parallel jobs (one per architecture), each independently calling svenstaro/upload-release-action. This caused a race condition where multiple releases were created for the same tag (we saw 3 duplicate 0.31.0 releases).
  • Removed the matrix from publish, using download-artifact with pattern + merge-multiple: true to download all artifacts in a single job.
  • Merged publish-assets into the publish job so only one job creates the release.

Test plan

  • Push a test tag to verify a single release is created with all assets

The test release https://github.com/dfinity/sdk/releases/tag/0.0.0-test.0 contains all 20 artifacts as expected.

The publish job used a matrix strategy that ran 4 parallel jobs (one per
architecture), each independently calling upload-release-action. This
caused a race condition where multiple releases were created for the
same tag. Fix by removing the matrix, downloading all artifacts in a
single job with merge-multiple, and merging the publish-assets job into
the publish job.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lwshang lwshang requested a review from a team as a code owner March 3, 2026 17:36
@lwshang lwshang enabled auto-merge (squash) March 3, 2026 19:26
@lwshang lwshang merged commit da69f3e into master Mar 4, 2026
47 checks passed
@lwshang lwshang deleted the fix/publish-workflow-duplicate-releases branch March 4, 2026 00:03
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