-
Notifications
You must be signed in to change notification settings - Fork 186
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
Adds support for multi-platform linux/amd64 & linux/arm64 docker images #603
Conversation
fd5e7d5
to
03eea53
Compare
docker-compose-internal.yaml
Outdated
@@ -24,6 +24,9 @@ services: | |||
build: | |||
context: . | |||
dockerfile: node/cmd/Dockerfile | |||
platforms: | |||
- "linux/amd64" | |||
- "linux/arm64" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add arm images to the other components too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dropping this from this change. Building with docker compose is kludgy and we might be better off using buildx directly. This is also triggering a failure with gitversion that needs more debugging.
This reverts commit 8c82532.
This change will allow operators to run released docker images on ARM based AWS Gravitron instances as well as M1/2/3 Macs
Multi-platform docker compose builds need to be built + pushed within the same context, so this change consolidates the previous 2-staged built + push flow into a single build + potentially push flow via a
PUSH_FLAG
env variable that gets set if a push is intended.Why are these changes needed?
Checks