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

fix: bundle correct arch for the mindthegap binary #954

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

dkoshkin
Copy link
Contributor

What problem does this PR solve?:
Latest changes broke the published arm build. The mindthegap binary from bundle_builder will always be BUILDPLATFORM, but we want it to match TARGETPLATFORM.

Which issue(s) this PR fixes:
Fixes #

How Has This Been Tested?:

Special notes for your reviewer:

@dkoshkin dkoshkin force-pushed the dkoshkin/fix-arm-mindthegap-image branch from 6a76eb3 to f56a76a Compare October 29, 2024 03:20
@dkoshkin dkoshkin requested a review from faiq October 29, 2024 17:49
@dkoshkin dkoshkin merged commit 9d922aa into main Oct 29, 2024
20 checks passed
@dkoshkin dkoshkin deleted the dkoshkin/fix-arm-mindthegap-image branch October 29, 2024 19:05
@dkoshkin dkoshkin mentioned this pull request Oct 29, 2024
dkoshkin added a commit that referenced this pull request Oct 29, 2024
🤖 I have created a release *beep* *boop*
---


## 0.21.0 (2024-10-29)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

## What's Changed
### Exciting New Features 🎉
* feat: update Nutanix CSI to 3.1.0 by @dkoshkin in
#956
### Fixes 🔧
* fix: bundle correct arch for the mindthegap binary by @dkoshkin in
#954
* fix: use correct securityContext for Alpine based helm-repository Pod
by @dkoshkin in
#955


**Full Changelog**:
v0.20.0...v0.21.0

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants