-
Notifications
You must be signed in to change notification settings - Fork 53
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 to v0.6.0 and add port mapping support. #1
Conversation
@mrunalp Do you prefer we maintain this in our repo or get this merged here? :) |
@Random-Liu This looks fine to me 👍 :) |
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.
Looks good to me.
ocicni.go
Outdated
// Search for vendor-specific plugins as well as default plugins in the CNI codebase. | ||
vendorDir := vendorCNIDir(vendorCNIDirPrefix, conf.Network.Type) | ||
vendorDir := vendorCNIDir(vendorCNIDirPrefix, confList.Plugins[0].Network.Type) |
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.
We should probably log which network executable we finally picked up and from which file. Will be helpful in debugging as we now have multiple files and multiple plugins in same file.
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.
Will do.
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.
Done.
Signed-off-by: Lantao Liu <lantaol@google.com>
3bb4225
to
0993d98
Compare
LGTM |
@mrunalp Thanks! I'll update the dependency in cri-containerd. Actually cri-o could also use this to get rid of the annoying hostport package. :p |
This PR:
@mrunalp
Signed-off-by: Lantao Liu lantaol@google.com