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: enable arm64 operator build #803
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
89e8446
to
fae7c1c
Compare
/cc @akrejcir |
Enable developer building operator on an arm64 machine as an additional option. Jira-Url: https://issues.redhat.com/browse/CNV-36775 Signed-off-by: Ben Oukhanov <boukhanov@redhat.com>
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
@@ -3,7 +3,7 @@ FROM registry.access.redhat.com/ubi9/ubi-minimal as builder | |||
|
|||
RUN microdnf install -y make tar gzip which && microdnf clean all | |||
|
|||
RUN curl -L https://go.dev/dl/go1.20.11.linux-amd64.tar.gz | tar -C /usr/local -xzf - |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@akrejcir @0xFelix How do you suggest to check if it's arm64 or amd64 here? Isn't architecture usually based on base images? e.g. Golang amd64 [1] or arm64 [2].
[1] https://hub.docker.com/r/amd64/golang
[2] https://hub.docker.com/r/arm64v8/golang
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There may be some way to detect which architecture is used and download the right package based on that.
This was the reason why I dropped my original PR. I didn't want to spend too much time figuring this out, if no one would use it.
We stopped using golang
images from docker hub, because there was a rate limit on them (0fd2e73), and now we probably don't want to depend on docker.
We cannot simply install golang
package using microdnf
, because a package with the needed version may not yet be available. We hit this problem in: #335 .
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
I'm closing this PR because it's not needed - I'm using VM to build SSP. |
What this PR does / why we need it:
Enable developer building operator on an arm64 machine as an additional option.
Jira-Url: https://issues.redhat.com/browse/CNV-36775
Which issue(s) this PR fixes:
Fixes #802
Release note: