feat: per-commit release artifacts instead of mere moving tag#13876
Conversation
|
This pull request has been marked as stale because of no activity in the last 30 days. It will be closed in the next 5 days unless it is tagged "no stalebot" or other activity occurs. |
d70fc48 to
797058f
Compare
|
This pull request has been marked as stale because of no activity in the last 30 days. It will be closed in the next 5 days unless it is tagged "no stalebot" or other activity occurs. |
The moving tag being `vX.Y.Z-main`, a tag for both helm chart and docker images. Also, this is a fix: removes a race that could lead to vX.Y.Z-main pointing to an older commit than it should. Signed-off-by: David L. Chandler <[email protected]>
5996f87 to
90a49ab
Compare
There was a problem hiding this comment.
Pull request overview
Updates the release GitHub Actions workflow so pushes to main publish per-commit (per-SHA) Helm chart and container image tags, and only “promote” the floating vX.Y.Z-main tags from the workflow run that is still at origin/main HEAD to avoid tag races.
Changes:
- Add a separate
floating_versionoutput and computeVERSIONformainas${floating_version}-${short_sha}to avoid tag collisions between parallelmainruns. - Gate floating Helm chart publication on the commit still being
origin/mainHEAD. - Promote floating container image tags via
docker buildx imagetools createonly when the commit is stillorigin/mainHEAD.
Signed-off-by: David L. Chandler <[email protected]>
| # origin/main advances between the two jobs and leaves the floating chart tag behind the | ||
| # floating image tag. This job needs [setup, goreleaser], so both the per-sha charts | ||
| # (pushed above) and the per-sha images (from goreleaser) already exist by now. | ||
| - name: Promote floating chart and image tags if this commit is still main HEAD |
There was a problem hiding this comment.
I think that this should run as a nightly job that promotes the latest tag instead of on a per push to main
Description
The moving tag being
vX.Y.Z-main, a tag for both helm chart and docker images.Also, this is a fix: removes a race that could lead to vX.Y.Z-main pointing to an older commit than it should.
ghcr.io cost doesn't increase because public packages on GHCR have no storage or bandwidth quota.
Change Type
/kind feature
Changelog
Additional Notes