-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
images/cilium: set IMAGE_CROSS_TARGET_PLATFORM for right arch #15074
Conversation
Looks like we need a cross arch |
7977a7f
to
d98fbb4
Compare
d98fbb4
to
151523f
Compare
test-me-please |
151523f
to
2ea28e7
Compare
test-me-please |
retest-gke |
retest-1.20-4.9 (previous https://jenkins.cilium.io/job/Cilium-PR-K8s-1.20-kernel-4.9/753/) |
2ea28e7
to
363b253
Compare
363b253
to
34f5599
Compare
test-me-please |
Signed-off-by: André Martins <andre@cilium.io>
This variable is used to cross compile certain targets. Fixes the build of 'cilium-map-migrate' in multi-arch. Fixes: 706fc42 ("add multi-arch support into all images") Signed-off-by: André Martins <andre@cilium.io>
34f5599
to
dfca1a3
Compare
test-me-please |
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.
For the builder image side, this LGTM. We seem to add the arm64 compiler to the x86 image as well, but since we won't ship this image to user clusters I'm fine with that.
Looking at commit 68c961d however, we also recently removed the compilers image from the runtime side. What I'm expecting at a high level is that we build our own version of LLVM, strip it down so it only contains the bits necessary for BPF compilation, and we ship that compiler in the runtime image.
This variable is used to cross compile certain targets. Fixes the build
of 'cilium-map-migrate' in multi-arch.
Fixes: 706fc42 ("add multi-arch support into all images")
Signed-off-by: André Martins andre@cilium.io
Fixes: #15073