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

Pull the istio images from the official mirror to GCR #4022

Merged
merged 1 commit into from
May 11, 2021
Merged

Pull the istio images from the official mirror to GCR #4022

merged 1 commit into from
May 11, 2021

Conversation

ialidzhikov
Copy link
Member

/kind enhancement

We face dockerhub rate limit issues for istio images in environments where we don't use imagevector overwrite:

  Warning  Failed            6m48s (x4 over 10m)  kubelet             Failed to pull image "docker.io/istio/pilot:1.8.0": rpc error: code = Unknown desc = Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

It seems to tackle exactly this problem the istio community also provides official mirrors to Google Container Registry.

Ref https://istio.io/latest/blog/2020/docker-rate-limit/

Release note:

istio images are now pulled from the official mirrors to [Google Container Registry](https://gcr.io/istio-release) to prevent any potential dockerhub rate limit issues in environments that use the default images (don't specify any imagevector overwrite).

@ialidzhikov
Pull the istio images from the official mirror to GCR
944f144 
Signed-off-by: ialidzhikov <i.alidjikov@gmail.com>
@ialidzhikov ialidzhikov requested a review from a team as a code owner May 11, 2021 07:23
@gardener-robot gardener-robot added kind/enhancement Enhancement, improvement, extension needs/review size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 11, 2021
@ialidzhikov
Copy link
Member Author

/assign @mvladev

vpnachev
vpnachev approved these changes May 11, 2021
View reviewed changes
Copy link
Member

@vpnachev vpnachev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@vpnachev vpnachev merged commit 3399d25 into master May 11, 2021
@vpnachev vpnachev deleted the rc branch May 11, 2021 07:41
krgostev pushed a commit to krgostev/gardener that referenced this pull request Apr 21, 2022
@ialidzhikov
Pull the istio images from the official mirror to GCR (gardener#4022)
c4eaeef 
Signed-off-by: ialidzhikov <i.alidjikov@gmail.com>
krgostev pushed a commit to krgostev/gardener that referenced this pull request Jul 5, 2022
@ialidzhikov
Pull the istio images from the official mirror to GCR (gardener#4022)
67ec5ed 
Signed-off-by: ialidzhikov <i.alidjikov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vpnachev vpnachev vpnachev approved these changes

@mvladev mvladev mvladev approved these changes

Assignees

@mvladev mvladev

Labels
kind/enhancement Enhancement, improvement, extension size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@ialidzhikov @vpnachev @mvladev @gardener-robot-ci-1 @gardener-robot-ci-3 @gardener-robot