-
Notifications
You must be signed in to change notification settings - Fork 788
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
conformance tests: use mirror.gcr.io for most images #5673
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nalind 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 |
LGTM |
I'm confused. There's already a |
hold on, I merged local-registry in #5584. We should absolutely not be using any network pulls any more! What's happening? |
Conformance tests use |
Well, not |
dfb07ad
to
2f1be3c
Compare
Use busybox and alpine images from mirror.gcr.io, where possible, to avoid tripping pull limits in CI. Signed-off-by: Nalin Dahyabhai <[email protected]>
2f1be3c
to
05da9a7
Compare
/lgtm |
What type of PR is this?
/kind failing-test
What this PR does / why we need it:
Use busybox and alpine images from mirror.gcr.io, where possible, to avoid tripping pull limits in CI.
How to verify it
So many updated conformance tests!
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?