-
Notifications
You must be signed in to change notification settings - Fork 296
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
🌱 Update Calico in e2e tests to v3.24.0 #1606
🌱 Update Calico in e2e tests to v3.24.0 #1606
Conversation
The manifest is taken from https://raw.githubusercontent.com/projectcalico/calico/v3.24.0/manifests/calico.yaml. Signed-off-by: Johanan Liebermann <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use quay.io prefix URLs for images instead of the docker.io URLs since they are rate limited.
Docker Hub is rate limited. Signed-off-by: Johanan Liebermann <[email protected]>
Good point, thanks. I can see that the Calico maintainers have switched to quay.io as the official registry lately. Looks like they're going to update the manifests soon, too, so hopefully next time we shouldn't have to manually patch the official manifests. For the record, here is what I used to switch to Quay in this PR:
|
The registry has been updated to quay.io @srm09. |
/hold |
/retitle 🌱 Update Calico in e2e tests to v3.24.0 |
looks like the test failure is unrelated? probably don't have permissions to do that. |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: srm09 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 |
/unhold |
The broken CI is indeed unrelated to this change (see my comment above). We are waiting for #1602 to get merged. |
/retest |
What this PR does / why we need it: This PR updates the Calico manifest used by the e2e tests to
v3.24.0
. It also fixes #1605.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #1605
Special notes for your reviewer:
This is a large diff. To ensure it is up to date with upstream, run the following:
To test this PR, follow the reproduction instructions of #1605 and ensure the pods converge.
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
Release note: