fix: use gh release upload instead of softprops/action-gh-release#3840
Conversation
softprops/action-gh-release@v3 has deduplication logic that picks the wrong draft release when multiple drafts exist for the same tag (e.g. when a tag is moved). This caused crds.yaml, olm.yaml, and install.sh to be uploaded to a stale draft from a prior failed run rather than the current GoReleaser-created release, as seen in v0.43.0. Replace it with 'gh release upload', which uploads directly to the release associated with the tag without creating any new release objects. --clobber makes reruns safe. Assisted-by: Claude Signed-off-by: Todd Short <tshort@redhat.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the release workflow to avoid incorrect artifact uploads when multiple draft releases exist for the same tag (e.g., after a tag move). It replaces softprops/action-gh-release@v3 with gh release upload so assets are uploaded directly to the release associated with the tag, with --clobber enabling safe reruns.
Changes:
- Replace
softprops/action-gh-release@v3withgh release uploadfor uploading rendered quickstart manifests. - Add
--clobberto make uploads idempotent across reruns. - Authenticate GitHub CLI via
GH_TOKENusing the workflowgithub.token.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: grokspawn, rashmigottipati The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Since this PR includes a pivot away from the action, it kinda answers my earlier question about whether we should open an issue for softprops/action-gh-release@v3 (or PR it) to correct the behavior. I wonder if others are impacted by it. Edit: looks like others experience the bug too... softprops/action-gh-release#772 |
387b952
into
operator-framework:master
softprops/action-gh-release@v3 has deduplication logic that picks the wrong draft release when multiple drafts exist for the same tag (e.g. when a tag is moved). This caused crds.yaml, olm.yaml, and install.sh to be uploaded to a stale draft from a prior failed run rather than the current GoReleaser-created release, as seen in v0.43.0.
Replace it with 'gh release upload', which uploads directly to the release associated with the tag without creating any new release objects. --clobber makes reruns safe.
Assisted-by: Claude
Reviewer Checklist
/doc[FLAKE]are truly flaky and have an issue