-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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 cni spec to 0.4.0 (for net plugins allowing it) available since cni plugins 0.8.0 #6053
Conversation
58ddfa0
to
640bbc2
Compare
f0ca1e6
to
53271f2
Compare
/lgtm |
Weird, CNI spec 0.4.0 was released a year ago and very few CNI plugins support it. |
Yes I was hopeful before testing the matrix but in fact it's not that widespread at all :/ |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: floryut, Miouge1 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 |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Update cni spec for network plugin that allow it
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
CNI plugin spec matrix :
Also fix networking test error (if debug where executing on other instance than [0])
Does this PR introduce a user-facing change?: