-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Allow building storage-provisioner for other arch #6257
Allow building storage-provisioner for other arch #6257
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afbjorklund The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #6257 +/- ##
=======================================
Coverage 38.23% 38.23%
=======================================
Files 123 123
Lines 8269 8269
=======================================
Hits 3162 3162
Misses 4696 4696
Partials 411 411 |
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.
Instead of having a Dockerfile per architecture, could we pass in GOARCH as a build arg to docker build
?
Then we could have one Dockerfile that looks like:
FROM scratch
ARG arch
COPY out/storage-provisioner-$(arch) storage-provisioner
CMD ["/storage-provisioner"]
and one docker build command:
docker build -t $(REGISTRY)/storage-provisioner:$(STORAGE_PROVISIONER_TAG) -f deploy/storage-provisioner/Dockerfile --build-arg arch=$(GOARCH) .
I think that will work, would be good to add Can't really verify whether that (scratch) works on the mainframe, but... it's a static binary ? |
My guess is it would work? But that's just a guess 😅 |
I think you have to be really interested, to set up a test machine for I would be surprised if it was more than a binary number of minikube users on it ? And even if minikube works ok on arm/arm64, we don't really support the desktop. |
Also allow building the static binary in docker
c0206ac
to
bfa7c13
Compare
@priyawadhwa : syntax was close enough, Dockerfile uses |
For #6007
Also allow building the static binary in docker
By setting GOARCH, you can cross-compile.
The default is to build for the local architecture.