-
Notifications
You must be signed in to change notification settings - Fork 472
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
conformance: support non-amd64 architectures #1295
Comments
I think the source is https://github.com/kubernetes-sigs/ingress-controller-conformance/tree/master/images/echoserver - It should be easy to change and I am happy to do it, just not sure how to actually get it published |
/assign @howardjohn I'll need help getting it published officially though |
Hmm, interesting that we had found this to work in practice previously without explicitly building arm64 images - not quite sure how it was working without digging deeper, but hashicorp/learn-consul-kubernetes#65 was the behavior we found where switching to this image was working on an M1 chip. |
possibly using emulation?
…On Mon, Aug 22, 2022, 11:50 AM Mike Morris ***@***.***> wrote:
Hmm, interesting that we had found this to work in practice previosuly
without explicitly building arm64 images - not quite sure *how* it was
working without digging deeper, but hashicorp/learn-consul-kubernetes#65
<hashicorp/learn-consul-kubernetes#65> was the
behavior we found where switching to this image was working on an M1 chip.
—
Reply to this email directly, view it on GitHub
<#1295 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEYGXJKQQWEC6UNMNBK4L3V2PDXNANCNFSM55TBW2IA>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Similar to #627, we should publish all images used in conformance tests with multi-arch support.
One thing that confused me is @mikemorris seemed to suggest we picked the image we use because it supports arm64: kubernetes/kubernetes#110605 (comment). However, I do not see the same:
The text was updated successfully, but these errors were encountered: