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

add docker multiarch build #753

Merged

Conversation

ganievs
Copy link
Contributor

@ganievs ganievs commented Feb 28, 2024

Description of your changes

Add Docker multi-arch build

Fixes #712

I have:

  • Read and followed terraform-docs' [contribution process].
  • All tests pass when I run make test.

Signed-off-by: Shamil Ganiev <ganiev@pm.me>
@rickardp
Copy link

What is left for this to be merged?

Signed-off-by: Ganiev Shamil <ganiev@pm.me>
@pascal-hofmann
Copy link
Contributor

pascal-hofmann commented Sep 2, 2024

It looks like this changes the tags from latest to latest-<arch>, etc.

I think it would be nicer to push multi arch images with the existing tags instead.

Edit: I have no idea how goreleaser works, but I guess this might help: https://goreleaser.com/cookbooks/multi-platform-docker-images/#creating-multi-platform-docker-images-with-goreleaser

@ganievs
Copy link
Contributor Author

ganievs commented Sep 3, 2024

Hi @pascal-hofmann! The tag is specified a target architecture and it works seamlessly for the end user. You'll continue to pull in the same way by image:tag without an prefix

Copy link
Contributor

@pascal-hofmann pascal-hofmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pascal-hofmann pascal-hofmann merged commit d00cf13 into terraform-docs:master Sep 4, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

create ARM based container
4 participants