Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bump github.com/containernetworking/cni v1.1.0 #98

Merged
merged 1 commit into from Apr 27, 2022
Merged

bump github.com/containernetworking/cni v1.1.0 #98

merged 1 commit into from Apr 27, 2022

Conversation

aojea
Copy link
Contributor

@aojea aojea commented Apr 27, 2022 •
edited

This is a minor update to the CNI libraries and tooling. This does not bump the protocol / spec version, which remains at v1.0.0.

Changes include:

libcni: handle empty version when parsing version (containernetworking/cni#893). Without this, Delete failed for empty-version configs, which was a regression from v0.8.0.
Fix incorrect pointer inputs to json.Unmarshal (containernetworking/cni#880).
[exec-plugins]: support plugin lists

(containernetworking/cni#865).
skel: remove superfluous err nil check in (*dispatcher).pluginMain (containernetworking/cni#864).
skel: print out CNI versions supported in help text (containernetworking/cni#860).

@aojea
Copy link
Contributor Author

aojea commented Apr 27, 2022

/assign @mikebrow
/cc @squeed @dcbw

@aojea aojea force-pushed the cni branch 2 times, most recently from 8448c77 to 0c75cce Compare April 27, 2022 21:21
mikebrow
mikebrow approved these changes Apr 27, 2022
View reviewed changes
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MikeZappa87
Copy link

LGTM

@aojea
bump github.com/containernetworking/cni v1.1.0
e24193e 
Signed-off-by: Antonio Ojea <antonio.ojea.garcia@gmail.com>
@mikebrow mikebrow merged commit b39860d into containerd:main Apr 27, 2022
@aojea aojea mentioned this pull request May 3, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikebrow mikebrow mikebrow approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aojea @MikeZappa87 @mikebrow