-
Notifications
You must be signed in to change notification settings - Fork 585
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
upgrade cni plugin version to use new feature #1323
Conversation
I think it's probably OK to get updates to the CNI requirements, but... ...question -- does this really address the issue? like, if those changes are in the resulting binaries, does that address your issue if it's just the libraries updated in Multus? |
Based on your comment, you want to have fix for
This parameter is added in |
Yeah, those binaries aren't automatically included with Multus. Multus just uses some specific functionality from the plugins repo (we think for netns utilities). You would build the binaries from the containernetworking/plugins repo, and then install them onto the hosts 👍 |
I think this cnilib bump might be bigger than just a bump and might require some functional changes in the tests, so, let's close this out for now. Thanks for the contribution and asking the question |
Here is my config. Does that mean I should update
|
@dougbtv Thanks for you info. If I need update cni-plugin on my host. Can you kindly provide more info, on how to upgrade cni to my host? Thanks very much. |
upgrade
github.com/containernetworking/plugins v1.1.0 to v1.5.1
to use new features. Like below:
containernetworking/plugins@821982d
This new parameter allows users to remove the default vlan
Fixes: containernetworking/plugins#667