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

CI: push multiarch Docker images #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

squat
Copy link

@squat squat commented Jan 3, 2022

This commit modifies the CI configuration so that the containers that
are built are actually manifest lists with images for multiple
architectures, not just AMD64. This is helpful for modern clouds and
hybrid clusters, where it's possible that not all machines are AMD64.

I think this should work out of the box, since the s3_exporter binaries
are built in the container itself, and the golang images are all multiarch.

Signed-off-by: Lucas Servén Marín [email protected]

This commit modifies the CI configuration so that the containers that
are built are actually manifest lists with images for multiple
architectures, not just AMD64. This is helpful for modern clouds and
hybrid clusters, where it's possible that not all machines are AMD64.

Signed-off-by: Lucas Servén Marín <[email protected]>
@tboeghk
Copy link

tboeghk commented Jul 5, 2022

I desperately needed the s3_exporter for arm64 architecture so I used your PR and forked the project to observabilitystack/s3_exporter.

arm64 and amd64 images are now published to Docker Hub and the GitHub Container Registry.

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