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.

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

Fix calico manifest URL for v3.24.0 and newer #617

Closed
wants to merge 2 commits into from

Conversation

danifr
Copy link
Contributor

@danifr danifr commented Feb 16, 2023

From v3.24.0 and newer, Calico changed the location of the URL.
3.25 is the latest stable version.

Fix #584

Abort calico installation if cni_provider_version is not set
@danifr danifr requested a review from a team as a code owner February 16, 2023 21:35
@CLAassistant
Copy link

CLAassistant commented Apr 19, 2023

CLA assistant check
All committers have signed the CLA.

@jordanbreen28
Copy link
Contributor

jordanbreen28 commented Jun 12, 2023

Thanks @danifr for your work on this!
This work was covered in a separate PR that was already merged (#625), so voting to close. But this submission was greatly appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Calico install doesn't work
4 participants