-
Notifications
You must be signed in to change notification settings - Fork 122
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 dependencies to v1.0.0 #351
Conversation
Looks like I'll have to update the same in https://github.com/awslabs/tc-redirect-tap/ as well. |
@darkowlzz Can you update CNI Plugins to 1.0.1 which is the official release. The 1.0.0 release was broken. |
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.
Update to v1.0.1 which fixes the SPEC version bug.
`IPConfig` in v1.0.1 removes the `Version` field. Signed-off-by: Sunny <darkowlzz@protonmail.com>
Updated to CNI dependencies to v1.0.1. Thanks @gaby . |
I already submitted 1.0.1 for awslabs/tc-redirect-tap#11 |
@kzys Friendly bump, this PR is related to awslabs/tc-redirect-tap#11 |
@kzys, any news on getting this merged? Thanks! |
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.
LGTM. Thanks!
Updates CNI dependencies to v1.0.0.
IPConfig
in v1.0.0 removes theVersion
field.In the ignite project, we would like to use CNI v1.0.0, but since ignite uses firecracker-go-sdk, CNI dependencies have to be updated here as well. The change is mostly based on containernetworking/cni#783 . Hope that's okay.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.