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.

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

Arm64 multi-arch support and update documentation #1218

Closed
scottmalkie opened this issue Sep 15, 2020 · 3 comments
Closed

Arm64 multi-arch support and update documentation #1218

scottmalkie opened this issue Sep 15, 2020 · 3 comments
Labels
bug calico Calico integration issue documentation

Comments

@scottmalkie
Copy link

What happened:
The calico.yaml as referenced in the EKS documentation fails on Arm64 nodes, as the quay.io images are not multi-arch, and there is no logic to determine the underlying architecture in the yaml.

What you expected to happen:
The calico.yaml both in the repo and referenced in the docs would work on Arm64 nodes, now the EKS support for Graviton2 is GA, and calico itself works well on Arm64.

How to reproduce it (as minimally and precisely as possible):
Create EKS cluster, create an Arm64-based managed node group, attempt to deploy Calico via the documentation.

Anything else we need to know?:
Happy to help fix if you'd like, it should be pretty simple to add os/arch selectors.
If this is going to be a while to fix, if the docs could be updated in the meantime so as not to cause customer distress.
Also as the Stars Policy Demo is still not officially supported on Arm64, please update the docs to mention that, for customers who are following the docs with Arm64 clusters.

There was a previous here: #1047, however it only covered the preview (and the official docs haven't been updated post-EKS Graviton2 GA)

@mogren mogren added the calico Calico integration issue label Sep 15, 2020
@fawadkhaliq
Copy link

We should consolidate this YAML or have one source of truth for generating the Calico YAML (and others for that matter) and push to Helm charts. There's an Helm chart with the same content here: https://github.com/aws/eks-charts/tree/master/stable/aws-calico. I will sync up @jaypipes on how we can to manage these and report back

@mogren
Copy link
Contributor

mogren commented Sep 23, 2020

Also keep this issue in mind #876

@jayanthvn
Copy link
Contributor

Hi @scottmalkie

#1282 is merged which supports ARM and AMD. Will pick the fix in next release.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug calico Calico integration issue documentation
Projects
None yet
Development

No branches or pull requests

4 participants