Skip to content

Commit 949ede5

Browse files
committed
fix(ci): attach assets when release changed form draft to published
1 parent d7dcf6a commit 949ede5

1 file changed

Lines changed: 12 additions & 4 deletions

File tree

.github/workflows/release-assets.yml

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,23 @@
11
name: Upload release assets
22

3-
# When a GitHub release is created for a tag, render the install manifests for
4-
# that tag's image and attach them to the release. `make build-dist-manifests`
5-
# is `helm template` of the chart with image.repository/tag set to the released
3+
# When a GitHub release is published, render the install manifests for that
4+
# tag's image and attach them to the release. `make build-dist-manifests` is
5+
# `helm template` of the chart with image.repository/tag set to the released
66
# ref; the chart renders image == OPERATOR_IMAGE, so the attached YAML deploys
77
# the matching operator and its snapshot/restore agent. For consumers who
88
# kubectl-apply rather than helm-install.
9+
#
10+
# Trigger is `published`, NOT `created`: the intended flow is release-drafter
11+
# maintaining a draft that a maintainer then publishes, and the `created`
12+
# activity type does NOT fire when a draft is published (only `published`
13+
# does). With `created` the assets silently never attach for any draft-based
14+
# release. `published` also covers a release created already-public. The
15+
# manual `workflow_dispatch` (run against the tag ref) is the backfill path
16+
# for a release that was published before this fix.
917

1018
on:
1119
release:
12-
types: [ created ]
20+
types: [ published ]
1321
workflow_dispatch:
1422

1523
env:

0 commit comments

Comments
 (0)