Skip to content

Produce stable image suffix on release-14 merges (symmetric with release-13) - #229

Merged
ibrahim halatci (ihalatci) merged 1 commit into
release-14from
ihalatci-symmetric-release-14-images
Jul 1, 2026
Merged

Produce stable image suffix on release-14 merges (symmetric with release-13)#229
ibrahim halatci (ihalatci) merged 1 commit into
release-14from
ihalatci-symmetric-release-14-images

Conversation

@ihalatci

Copy link
Copy Markdown
Contributor

Make the release-14 build-test-images workflow symmetric with release-13.

Problem: On release-13 the release image jobs fire for any release-* branch, so merging into release-13 mints a stable -v<sha> suffix. On release-14 the jobs are gated to master only, so merging into release-14 produced only -dev-<sha> images (no stable -v suffix) — e.g. #228 merged and yielded only -dev-877ec08.

Fix (surgical, 4 lines): widen the branch guards so release-* merges route to the release jobs while feature/PR branches keep producing -dev images:

  • build_dev_images_per_postgres / build_dev_images_shared: != 'master'!= 'master' && !startsWith(github.ref_name, 'release-')
  • build_release_images / build_release_images_shared: == 'master'== 'master' || startsWith(github.ref_name, 'release-')

The two guards remain mutually exclusive, so no job double-fires. Behaviorally symmetric to release-13.

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Make the release-14 build-test-images workflow symmetric with release-13:
merging into any release-* branch (not just master) now fires the release
image jobs, minting a stable -v<sha> suffix instead of -dev-<sha>. Feature
and PR branches still publish -dev images.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ihalatci
ibrahim halatci (ihalatci) merged commit f4adc47 into release-14 Jul 1, 2026
15 checks passed
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.

2 participants