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

Store cilium chart for registry mirror in isovalent repository #1629

Closed
vivek-koppuru opened this issue Mar 25, 2022 · 3 comments
Closed

Store cilium chart for registry mirror in isovalent repository #1629

vivek-koppuru opened this issue Mar 25, 2022 · 3 comments
Milestone

Comments

@vivek-koppuru
Copy link
Member

Currently import-images stores the cilium chart under the registry of <registry-mirror>/cilium-chart, while the cilium image is stored under <registry-mirror>/isovalent/cilium. We should reuse the same isovalent repository instead of have the user have to create a separate repository called cilium-chart to decrease the number of repositories that have to be created.

We also seem to have different naming for other images, such as <registry-mirror>/l0g8r8j6/kubernetes-sigs/cluster-api/kubeadm-bootstrap-controller, so maybe we can use l0g8r8j6 as the common repository instead, or have a command that will create the necessary repositories for the user instead of creating them manually through these steps: https://anywhere.eks.amazonaws.com/docs/reference/clusterspec/registrymirror/#registry-configurations

@vivek-koppuru vivek-koppuru added this to the next+1 milestone Mar 25, 2022
@mrajashree
Copy link
Contributor

We should reuse the same isovalent repository instead of have the user have to create a separate repository called cilium-chart

👍

so maybe we can use l0g8r8j6 as the common repository instead

that registry only gets used for dev, but a command creating necessary repos would be good

@mrajashree
Copy link
Contributor

This PR covers the part of storing cilium chart under isovalent project. It gets pushed to isovalent/cilium, with separate tags for helm chart and docker image

@vivek-koppuru
Copy link
Member Author

Resolving this issue in favor of #1783

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants