Skip to content

Comments

Add latest tag to multi-platform manifest#564

Merged
NicoHinderling merged 4 commits intomainfrom
fix-latest-tagging-for-launchpad
Feb 9, 2026
Merged

Add latest tag to multi-platform manifest#564
NicoHinderling merged 4 commits intomainfrom
fix-latest-tagging-for-launchpad

Conversation

@NicoHinderling
Copy link
Contributor

Summary

  • ghcr.io/getsentry/launchpad:latest only had a linux/arm64 manifest because each arch build
    independently pushed latest, with arm64 winning the race
  • nightly worked correctly because it was included in the multi-platform manifest step, but latest was
    not
  • This broke devservices up --mode launchpad on amd64 machines

Changes - Set tag_latest: false on individual arch builds to stop the race condition

  • Added --tag ghcr.io/getsentry/launchpad:latest to the create-manifest step so latest gets a proper multi-platform manifest like nightly
    ## Verification
  • After CI runs, docker manifest inspect ghcr.io/getsentry/launchpad:latest should show both linux/amd64 and linux/arm64
  • devservices up --mode launchpad should work on both architectures

Copy link
Member

@hubertdeng123 hubertdeng123 left a comment

Choose a reason for hiding this comment

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

Can we just change https://github.com/getsentry/launchpad/blob/main/devservices/config.yml#L25 to use the nightly image? Latest is somewhat opinionated here since it's the tag we use to associate for self-hosted released images

@@ -42,7 +42,7 @@ jobs:
tag_suffix: -${{ matrix.platform }}
ghcr: true
tag_nightly: true
Copy link
Member

Choose a reason for hiding this comment

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

I think tag_nightly and tag_latest here should both be false, since we're tagging only in the multiplatform build

docker buildx imagetools create \
--tag ghcr.io/getsentry/launchpad:${{ github.sha }} \
--tag ghcr.io/getsentry/launchpad:nightly \
--tag ghcr.io/getsentry/launchpad:latest \
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we'd need the latest to be tagged here anymore

@NicoHinderling NicoHinderling force-pushed the fix-latest-tagging-for-launchpad branch from 9bed6ad to b6852ef Compare February 9, 2026 21:18
NicoHinderling and others added 2 commits February 9, 2026 13:21
CI resolved ruff>=0.14.9 to 0.15.0 which formats ternary
expressions in lambdas differently than 0.14.9.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@NicoHinderling NicoHinderling merged commit fcdd1d9 into main Feb 9, 2026
19 checks passed
@NicoHinderling NicoHinderling deleted the fix-latest-tagging-for-launchpad branch February 9, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants