Skip to content

fix(release): sequence release-pr after release and trigger binary build#184

Merged
zeitlinger merged 2 commits intomainfrom
fix/release-pr-ordering
Apr 17, 2026
Merged

fix(release): sequence release-pr after release and trigger binary build#184
zeitlinger merged 2 commits intomainfrom
fix/release-pr-ordering

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

@zeitlinger zeitlinger commented Apr 16, 2026

Summary

  • Release PR and Release jobs ran concurrently — Release PR computed the next version before the new tag existed, using the full commit history as changelog
  • Fix: needs: [release] ensures the tag is created before the next release PR is computed
  • GITHUB_TOKEN-created tags don't fire push: tags: triggers in other workflows — same limitation as release-please
  • Fix: dispatch release.yml directly from the release job when a release is created (same pattern as prometheus/client_java)

Test plan

  • Merge and verify the next release PR only contains commits since the last tag
  • Verify release.yml triggers automatically after the next release PR is merged

Release PR was computing the next version concurrently with the Release
job creating the git tag. If Release PR won the race, it had no new tag
as a baseline and included the full commit history in the changelog.

Adding needs: [release] ensures the tag exists before release-pr
determines what's changed since the last release.
Signed-off-by: Gregor Zeitlinger <[email protected]>
@zeitlinger zeitlinger marked this pull request as ready for review April 16, 2026 17:29
@zeitlinger zeitlinger requested a review from a team as a code owner April 16, 2026 17:29
Copilot AI review requested due to automatic review settings April 16, 2026 17:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ensures the Release PR job runs only after the release job completes so the newly created git tag exists before release-plz computes the next changelog baseline, preventing the “full history” changelog race on main.

Changes:

  • Add needs: [release] to the release-pr job to enforce ordering.
  • Add an inline comment documenting why the dependency exists.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zeitlinger zeitlinger enabled auto-merge (squash) April 16, 2026 17:31
…es tag

GITHUB_TOKEN-created tags don't fire push:tags: triggers in other
workflows. Use workflow_dispatch instead, same pattern as
prometheus/client_java's release-please setup.
Signed-off-by: Gregor Zeitlinger <[email protected]>
@zeitlinger zeitlinger changed the title fix(release): run release-pr after release to avoid race fix(release): sequence release-pr after release and trigger binary build Apr 16, 2026
Comment thread .github/workflows/release-plz.yml
@zeitlinger zeitlinger merged commit a2b314a into main Apr 17, 2026
14 checks passed
@zeitlinger zeitlinger deleted the fix/release-pr-ordering branch April 17, 2026 06:55
@github-actions github-actions Bot mentioned this pull request Apr 17, 2026
zeitlinger pushed a commit that referenced this pull request Apr 17, 2026
### Fixed

- *(release)* run release-pr after release to avoid race
([#184](#184))

> [!IMPORTANT]
> Close and reopen this PR to trigger CI checks.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants