Skip to content

Conversation

@aldy505
Copy link
Collaborator

@aldy505 aldy505 commented Jun 7, 2025

@aldy505 aldy505 requested a review from a team as a code owner June 7, 2025 01:50
@evanh
Copy link
Member

evanh commented Jun 9, 2025

/gcbrun

docker manifest create \
ghcr.io/getsentry/taskbroker:${{ github.sha }} \
ghcr.io/getsentry/taskbroker:latest \
Copy link
Member

Choose a reason for hiding this comment

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

All of our deploys using the latest image I believe. I'm not an expert on docker manifests: will this expose that image so deploys continue to work?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes. cc @Dav1dde

Copy link
Member

Choose a reason for hiding this comment

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

All of our deploys using the latest image I believe.

The deploys under the hood use a fixed sha, but also just the ones published to the google registry (not ghcr).

@evanh
Copy link
Member

evanh commented Jun 10, 2025

/gcbrun

@aldy505
Copy link
Collaborator Author

aldy505 commented Jun 10, 2025

@evanh @markstory what's blocking this PR?

@markstory
Copy link
Member

/gcbrun

@markstory markstory merged commit d32e924 into getsentry:main Jun 11, 2025
17 checks passed
@sentry-taskbroker-fast-revert-bot

PR reverted: 32f1e94

sentry-taskbroker-fast-revert-bot bot pushed a commit that referenced this pull request Jun 11, 2025
@markstory
Copy link
Member

I reverted this change as it failed to run https://github.com/getsentry/taskbroker/actions/runs/15588079414

@aldy505
Copy link
Collaborator Author

aldy505 commented Jun 12, 2025

Let me fix that in a few hours

@aldy505 aldy505 deleted the ci/provide-arm64-image branch June 12, 2025 00:39
@aldy505
Copy link
Collaborator Author

aldy505 commented Jun 12, 2025

second attempt here: #412

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.

4 participants