-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Use quay images for cilium #11722
Use quay images for cilium #11722
Conversation
@olemarkus Any thoughts about doing this the same way we do for Calico? |
Was considered and decided against: #10204 |
I guess we can revisit that. The decision was in the context of us doing an API change soon. |
Upstream default is a powerful argument. For kOps 1.22, local image registries will be a lot easier to manage. |
/lgtm |
I agree.
Still harder than setting the image version or registry. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hakman The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…722-origin-release-1.21 Automated cherry pick of #11722: Use quay images for cilium
a) again I got a request for this over docker rate limiting
b) Since with containerd, docker credentials are not valid for image pulls outside of kubelet, docker.io images will hit rate limiting
c) upstream use quay by default
cilium with hubble already has a dependency on cert-manager, which is hosted on quay so quay is to some extent a dependency anyway.