Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #345 - Build ARM64 image. #346

Merged
merged 4 commits into from
Sep 9, 2024
Merged

Conversation

jsuereth
Copy link
Contributor

@jsuereth jsuereth commented Aug 28, 2024

Follows the docker guide for setting up QEMU to build an ARM + x86 image.

Fixes #345

Note: This may not be the fastest way to do these builds, it's just the most convenient.

@jsuereth jsuereth requested a review from a team August 28, 2024 12:15
@jsuereth jsuereth changed the title Fix #245 - Build ARM64 image. Fix #345 - Build ARM64 image. Aug 28, 2024
@jsuereth
Copy link
Contributor Author

@lquerel It looks like building both in test takes a really long time. Do we think it's worth having the previous test-image build we had and assuming building for ARM in docker will succeed on release or should we perform it in every pull request?

@jsuereth
Copy link
Contributor Author

Oof - so here' s an alternative: https://docs.docker.com/build/ci/github-actions/multi-platform/#distribute-build-across-multiple-runners

Take a look and let me know what you think - I'll look into the more complicated setup a bit later.

@jsuereth
Copy link
Contributor Author

I updated this so the combined image is only created on release for now.

@lquerel lquerel merged commit cdea78f into open-telemetry:main Sep 9, 2024
21 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.

Publish docker image with for arm64 platform
2 participants