-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Cross-build hyperkube and debian-iptables for ARM. Also add a flannel image #21617
Conversation
Labelling this PR as size/L |
GCE e2e test build/test passed for commit 2d6c0ecf86c035d9f71a540d775d643049e9173b. |
The author of this PR is not in the whitelist for merge, can one of the admins add the 'ok-to-merge' label? |
9cef39e
to
3df5c7b
Compare
Labelling this PR as size/M |
GCE e2e test build/test passed for commit 9cef39ea0fe95741c96dd7e55db462ac14730168. |
GCE e2e test build/test passed for commit 3df5c7b471eb288ca31a8e3ee745f34cd5e3529f. |
Labelling this PR as size/L |
GCE e2e test build/test passed for commit f83aee048d191c6c26926a545cbaaa2bea97cca4. |
Not sure who's best equipped, but assigning to @david-mcmahon. @thockin might be good too. Very sorry for the delay, 1.2 was consuming all of my time and I'm just now going through my stack of PRs. |
@lavalamp Ok, what's your opinion about this? It's using Before this may be merged, some of you should push the It would be great to start with this, then I may create a follow-up PR that updates references to the images and so on. |
Unfortunately, I've never tried doing an arm cross-build, so I don't really have an opinion on this. But if you say it works I'll believe you :) |
As I don't plan on looking at this in the next couple of weeks at least, removing myself. @luxas feel free to take this and run with it. |
@david-mcmahon What do you mean? |
@luxas did you want to work on this? |
@k8s-bot test this please github issue: #IGNORE |
Thanks a lot @david-mcmahon! |
GCE e2e build/test passed for commit fab9ca549e89ba0d6fe5091b3bf8e443c600bc6b. |
LGTM pending passing of tests. |
…r arm, arm64 and ppc64le. Build and push hyperkube for arm on every release
Will update. Ran |
PR changed after LGTM, removing LGTM. |
Yay! Was able to get k8s up and running by following the official |
@luxas Thanks for taking the time to explain the need for bootstrapping the images. |
GCE e2e build/test passed for commit 560268e. |
The author of this PR is not in the whitelist for merge, can one of the admins add the 'ok-to-merge' label? |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit 560268e. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit 560268e. |
Automatic merge from submit-queue |
We have to be able to build complex docker images too on
amd64
hosts.Right now we can't build Dockerfiles with
RUN
commands when building for other architectures e.g. ARM.Resin has a tutorial about this here: https://resin.io/blog/building-arm-containers-on-any-x86-machine-even-dockerhub/
But it's a bit clumsy syntax.
The other alternative would be running this command in a Makefile:
and
Then the kernel will be able to differ ARM binaries from amd64. When it finds a ARM binary, it will invoke
/usr/bin/qemu-arm-static
first and letsqemu
translate the ARM syscalls to amd64 ones.Some code here: https://github.com/multiarch
WDYT is the best approach? If registering
binfmt_misc
in the kernels of the machines is OK, then I think we should go with that.Otherwise, we'll have to wait for resin's patch to be merged into mainline qemu before we may use the code I have here now.
@fgrzadkowski @david-mcmahon @brendandburns @zmerlynn @ixdy @ihmccreery @thockin