gardener / gardener-extension-registry-cache

Gardener extension controller which deploys pull-through caches for container registries.
Apache License 2.0
7 stars 18 forks source link

Replace use of `docker.io` with `public.ecr.aws` to avoid docker hub pull rate limit errors #228

Closed dimitar-kostadinov closed 1 month ago

dimitar-kostadinov commented 1 month ago

How to categorize this PR?

/area testing /kind enhancement

What this PR does / why we need it: The prow CI runs e2e tests on each PR and this results in 6 image pulls from docker.io. .Since docker hub's pull limit is 100 downloads per 6 hours per IP address, we more often see a You have reached your pull rate limit error, causing PR checks to fail. To reduce docker hub image pull, docker.io is replaced by public.ecr.aws registry.

Which issue(s) this PR fixes: N/A

Special notes for your reviewer: To avoid conflicts with registries in local setup Drop registry caches for eu.gcr.io and ghcr.io PR should be merged first.

Release note:

NONE
gardener-prow[bot] commented 1 month ago

LGTM label has been added.

Git tree hash: b8c307db53eac6dd0ec63da789df454c5653461f

gardener-prow[bot] commented 1 month ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ialidzhikov

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: - ~~[OWNERS](https://github.com/gardener/gardener-extension-registry-cache/blob/main/OWNERS)~~ [ialidzhikov] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment