Skip to content

feat: per-commit release artifacts instead of mere moving tag#13876

Merged
davidjumani merged 2 commits into
kgateway-dev:mainfrom
chandler-solo:chandler/maintagrace
Jul 13, 2026
Merged

feat: per-commit release artifacts instead of mere moving tag#13876
davidjumani merged 2 commits into
kgateway-dev:mainfrom
chandler-solo:chandler/maintagrace

Conversation

@chandler-solo

@chandler-solo chandler-solo commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

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

Now storing release artifacts for each commit to make it easier to try out cutting-edge releases. Previously we only stored vX.Y.Z-main, a moving tag. Also fixes a minor race that could cause that tag to point to an older commit than desired.

Additional Notes

@gateway-bot gateway-bot added kind/feature Categorizes issue or PR as related to a new feature. release-note labels Apr 18, 2026
@github-actions

Copy link
Copy Markdown

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.

@github-actions github-actions Bot added the stale Issues that are stale. These will not be prioritized without further engagement on the issue. label May 18, 2026
@chandler-solo
chandler-solo force-pushed the chandler/maintagrace branch from d70fc48 to 797058f Compare May 18, 2026 19:15
@github-actions github-actions Bot removed the stale Issues that are stale. These will not be prioritized without further engagement on the issue. label May 19, 2026
@github-actions

Copy link
Copy Markdown

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.

@github-actions github-actions Bot added the stale Issues that are stale. These will not be prioritized without further engagement on the issue. label Jun 18, 2026
@chandler-solo chandler-solo removed the stale Issues that are stale. These will not be prioritized without further engagement on the issue. label Jun 23, 2026
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]>
@chandler-solo
chandler-solo force-pushed the chandler/maintagrace branch from 5996f87 to 90a49ab Compare June 23, 2026 14:32
@chandler-solo
chandler-solo marked this pull request as ready for review June 23, 2026 15:33
Copilot AI review requested due to automatic review settings June 23, 2026 15:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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_version output and compute VERSION for main as ${floating_version}-${short_sha} to avoid tag collisions between parallel main runs.
  • Gate floating Helm chart publication on the commit still being origin/main HEAD.
  • Promote floating container image tags via docker buildx imagetools create only when the commit is still origin/main HEAD.

Comment thread .github/workflows/release.yaml
Comment thread .github/workflows/release.yaml Outdated
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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think that this should run as a nightly job that promotes the latest tag instead of on a per push to main

@davidjumani
davidjumani added this pull request to the merge queue Jul 13, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 13, 2026
@davidjumani
davidjumani added this pull request to the merge queue Jul 13, 2026
Merged via the queue into kgateway-dev:main with commit c275deb Jul 13, 2026
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature Categorizes issue or PR as related to a new feature. release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants