Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support arm64 cilium/json-mock #24170

Closed
kahirokunn opened this issue Mar 4, 2023 · 2 comments · Fixed by #24173 or #31373
Closed

Support arm64 cilium/json-mock #24170

kahirokunn opened this issue Mar 4, 2023 · 2 comments · Fixed by #24173 or #31373
Labels
area/build Anything to do with the build, more general then area/CI area/CI Continuous Integration testing issue or flake kind/feature This introduces new functionality.

Comments

@kahirokunn
Copy link
Contributor

Cilium Feature Proposal

Because cilium/json-mock does not support arm64, I get an error with the following yaml in my environment.

exec /bin/bash: exec format error

What about changing the cilium/json-mock image to a multi-arch image?
@kahirokunn kahirokunn added the kind/feature This introduces new functionality. label Mar 4, 2023
@sayboras
Copy link
Member

sayboras commented Mar 5, 2023

can you use the latest tag (e.g. v1.3.3) instead ? From v1.3.0, arm build is supported.

@sayboras sayboras mentioned this issue Mar 5, 2023
Merged
@sayboras sayboras added area/CI Continuous Integration testing issue or flake area/build Anything to do with the build, more general then area/CI labels Mar 5, 2023
sayboras added a commit to sayboras/cilium that referenced this issue Mar 10, 2023
@sayboras
chore: Update json-mock image
ca28703 
Just replace docker.io/cilium/json-mock.* to quay.io/cilium/json-mock:v1.3.3@sha256:f26044a2b8085fcaa8146b6b8bb73556134d7ec3d5782c6a04a058c945924ca0
Switching to quay.io is done as the current image in docker.io only
has amd64 build, but not arm version. It's also consistent with main
cilium image repository.

Fixes: cilium#24170
Signed-off-by: Tam Mach <tam.mach@cilium.io>
sayboras added a commit that referenced this issue Mar 10, 2023
@sayboras
chore: Update json-mock image
5c59e61 
Just replace docker.io/cilium/json-mock.* to quay.io/cilium/json-mock:v1.3.3@sha256:f26044a2b8085fcaa8146b6b8bb73556134d7ec3d5782c6a04a058c945924ca0
Switching to quay.io is done as the current image in docker.io only
has amd64 build, but not arm version. It's also consistent with main
cilium image repository.

Fixes: #24170
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@kahirokunn
Copy link
Contributor Author

Thx!

@loomkoom loomkoom mentioned this issue Mar 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/build Anything to do with the build, more general then area/CI area/CI Continuous Integration testing issue or flake kind/feature This introduces new functionality.
Projects
None yet
Milestone
No milestone
2 participants
@sayboras @kahirokunn