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

Build multi-arch k8s-helm image #89

Merged
merged 1 commit into from
Jun 1, 2021

Conversation

barthy1
Copy link
Contributor

@barthy1 barthy1 commented Feb 8, 2021

Use multi-arch build github actions for helm image, see https://github.com/docker/build-push-action#multi-platform-image
to get multi-arch image for linux/amd64, linux/arm64, linux/s390x, linux/ppc64le architectures

Also use docker buildx env variable(TARGETARCH) to properly identify target arch in
the Dockerfile https://docs.docker.com/engine/reference/builder/#automatic-platform-args-in-the-global-scope
amd64 is still default one and pure docker build will be done without any extra steps.

See Github action logs for master branch of fork repo here.

Use multi-arch build github action for helm image.
Also use docker buildx env variable to properly identify target arch in
the Dockerfile https://docs.docker.com/engine/reference/builder/#automatic-platform-args-in-the-global-scope

Signed-off-by: Yulia Gaponenko <yulia.gaponenko1@de.ibm.com>
@lachie83
Copy link
Owner

This is excellent. Thanks for the contribution. Will review (sorry for the delay)

@lachie83 lachie83 merged commit 8449d60 into lachie83:master Jun 1, 2021
@lachie83
Copy link
Owner

lachie83 commented Jun 1, 2021

I've merged this in. Please try out the latest OR v3.6.0 image to test the multi-arch images. Do you need this patch backported to a branch?

@barthy1
Copy link
Contributor Author

barthy1 commented Jun 1, 2021

yep, the latest version runs on s390x and ppc64le architectures 👍

Do you need this patch backported to a branch?

No, latest is enough, thank you.

@lachie83
Copy link
Owner

lachie83 commented Jun 1, 2021

Excellent. Thank you so much for your contribution and my apologies for taking so long to action.

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.

2 participants