-
Notifications
You must be signed in to change notification settings - Fork 326
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
Publish docker images for arm64 #4128
Publish docker images for arm64 #4128
Comments
@rootulp , can I take this one? |
go for it @tskoyo |
Hi team, here's the investigation:
When supporting multiple platforms, the output should be similar to the output from this example (see this part -> This limits users on ARM-based systems and results in either poor performance (via QEMU emulation) or inability to use the image. ReasonThe image was likely built using the default single-platform To support both
Risks
Read more about |
Thanks for the detailed write-up @tskoyo ! Our Docker images is published via this workflow: https://github.com/celestiaorg/celestia-app/actions/workflows/docker-build-publish.yml which seems defined via celestia-app/.github/workflows/docker-build-publish.yml Lines 20 to 28 in 0261d45
Based on this line I think it should build a Docker image that supports arm64 and amd64. Same with this log. Do you have any idea why it's not building an image that supports both of those platforms? |
@rootulp i believe the issue is here in these last 2 steps starting at line 283 https://github.com/celestiaorg/.github/blob/45533fc35847a8568deb0d4cae1851a2872f3651/.github/workflows/reusable_dockerfile_pipeline.yml#L283 The main issue is we build in two separate steps, but we re-use tags, so the manifest is overwritten by the amd64 build. Options:
|
Context
We're using a celestia-app Docker image in this docker-compose.yml
Problem
Proposal
2. If yes, why wasn't the docker-compose script able to import the arm64 version of the image?
3. if no, explore what changes need to be made to publish Docker images for both arm64 and amd64.
The text was updated successfully, but these errors were encountered: