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 arm64 builds #351

Merged
merged 3 commits into from
Nov 26, 2021
Merged

Add arm64 builds #351

merged 3 commits into from
Nov 26, 2021

Conversation

tylerauerbeck
Copy link
Contributor

Signed-off-by: Tyler Auerbeck [email protected]

What this PR does / why we need it:
This PR adds the building and publishing of arm64 images to quay when a release is published.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #342

Special notes for your reviewer:

This PR adds the publishing of arm64 images as well as amd64 images when a release is published. Because the same goreleaser functionality is used in the ci workflow, we needed to add the buildx and qemu actions there as well.

When a release is cut, there will now be a few images pushed to quay. For example, given a release of v3.4.5, you would end up with:

  • chart-testing:latest-amd64
  • chart-testing:latest-arm64
  • chart-testing:v3.4.5-amd64
  • chart-testing:v3.4.5-arm64
  • chart-testing:v3.4.5
  • chart-testing:latest

The last two would be the ones that would be useful to users and would direct them to the appropriate arch (and their manifests would just reference the underlying versions). Just wanted to call this out so it wasn't a surprise when this started to happen.

Signed-off-by: Tyler Auerbeck <[email protected]>
.goreleaser.yml Outdated Show resolved Hide resolved
.goreleaser.yml Outdated Show resolved Hide resolved
Signed-off-by: Tyler Auerbeck <[email protected]>
@paulczar
Copy link

paulczar commented Aug 5, 2021

hey @scottrigby can you confirm this looks good to you too ?

Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

looks like I need to catch up on goreleaser options!
Looked over docs, this all looks correct AFAICT. Let's trigger Ci…

@scottrigby
Copy link
Member

Also thank you @tylerauerbeck 😄

@scottrigby
Copy link
Member

CI failed. Can we check why? I'm not in a place to do that right now

@tylerauerbeck
Copy link
Contributor Author

@scottrigby Yepp. On my list to dig into today. I'll let you know what I find.

@tylerauerbeck
Copy link
Contributor Author

@scottrigby @paulczar Sorry for the delay on this one, but had some time to go back and take a look. I've been able to run this successfully on my own branch, which you can see here:

release: https://github.com/tylerauerbeck/chart-testing/actions/runs/1498224439
ci: https://github.com/tylerauerbeck/chart-testing/actions/runs/1498109036

Published images from the release workflow: https://quay.io/repository/tylerauerbeck/chart-testing?tab=tags

@cpanato
Copy link
Member

cpanato commented Nov 25, 2021

@tylerauerbeck please sign the dco

Signed-off-by: Tyler Auerbeck <[email protected]>
@tylerauerbeck
Copy link
Contributor Author

@cpanato Good catch. Signoff is there now.

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

thanks for that

@cpanato
Copy link
Member

cpanato commented Nov 26, 2021

@scottrigby will merge ok? :)

@cpanato cpanato merged commit 89dbd16 into helm:main Nov 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Arm Image
5 participants