-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
vendor: update github.com/containernetworking/cni to v0.7.2-0.20200304161608-4fae32b84921 #5414
vendor: update github.com/containernetworking/cni to v0.7.2-0.20200304161608-4fae32b84921 #5414
Conversation
…4161608-4fae32b84921 Specifically to get: containernetworking/cni#735 6f29b0165883b2b52ccd4dcb937162ea4c86927b intercept netplugin std err But also pulls in some interface name validation and a compatibility fix for configurations that don't set a CNI version. Signed-off-by: Dan Williams <dcbw@redhat.com>
/approve |
LGTM once tests go green |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dcbw, mheon 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 |
@mheon tests are green :) |
@baude @rhatdan @TomSweeneyRedHat @vrothberg @giuseppe PTAL and merge |
/lgtm 👼 |
Specifically to get:
containernetworking/cni#735
6f29b0165883b2b52ccd4dcb937162ea4c86927b intercept netplugin std err
But also pulls in some interface name validation and a compatibility
fix for configurations that don't set a CNI version.
Signed-off-by: Dan Williams dcbw@redhat.com