cri-o icon indicating copy to clipboard operation
cri-o copied to clipboard

contrib/cni/README.md suggests installing outdated CNI version 0.3.1, not supported by Kubernetes

Open dilyanpalauzov opened this issue 1 year ago • 4 comments

What happened?

According to https://kubernetes.io/docs/concepts/extend-kubernetes/compute-storage-net/network-plugins/ Kubernetes supports CNI-plugins with minimum specification version 0.4.0, but prefers version 1.0.0.

https://github.com/cri-o/cri-o/tree/main/contrib/cni#configuration-directory suggests copying files from cri-o.git/contrib/cni to /etc/cni/net.d. The files in cri-o.git/contrib/cni contain "cniVersion": "0.3.1".

Please elaborate why it is suggested to utilize the 0.3.1 interface, instead of a newer interface, supported by Kubernetes?

What did you expect to happen?

Not applicable.

How can we reproduce it (as minimally and precisely as possible)?

Not applicable.

Anything else we need to know?

Not applicable.

CRI-O and Kubernetes version

Not applicable.

OS version

Not applicable.

Additional environment details (AWS, VirtualBox, physical, etc.)

Not applicable.

dilyanpalauzov avatar Jul 20 '22 20:07 dilyanpalauzov

yeah we use 1.0.0 and greater now

haircommander avatar Jul 21 '22 20:07 haircommander

thanks for noticing all of these inconsistencies!

haircommander avatar Jul 21 '22 20:07 haircommander

A friendly reminder that this issue had no activity for 30 days.

github-actions[bot] avatar Aug 28 '22 00:08 github-actions[bot]

kubernetes-cni uses 0.8.7, which is less than 1.0.0

Currently that is sometimes leading to conflicts, with Kubic rpms.

containernetworking-plugins conflicts with kubernetes-cni

And it should still be OK with using 0.4.0, either way.

afbjorklund avatar Aug 29 '22 11:08 afbjorklund

"kubernetes-cni" was updated to 1.1.1, so should be fine

EDIT: or it will be, once the package has been fixed <sigh>

afbjorklund avatar Oct 09 '22 11:10 afbjorklund

The "kubernetes-cni" package still contains 0.8.6 (not supporting criVersion 1.0.0), but I guess cri-o comes with its own "containernetworking-plugins" package anyway... https://build.opensuse.org/project/show/devel:kubic:libcontainers:stable

afbjorklund avatar Dec 22 '22 15:12 afbjorklund

cri-o/contrib/cni/11-crio-ipv4-bridge.conflist contains "cniVersion": "0.3.1". Can this be reopened, until this is lifted to 1.0.0, too?

cri-o/contrib/cni/99-loopback.conflist has no plugins element, therefore it does not need .conflist file extension.

For the record, recently the 99-loopback file got a "name": "loopback" element, which was again recently removed.

dilyanpalauzov avatar Dec 23 '22 07:12 dilyanpalauzov

A friendly reminder that this issue had no activity for 30 days.

github-actions[bot] avatar Feb 03 '23 00:02 github-actions[bot]

Closing this issue since it had no activity in the past 90 days.

github-actions[bot] avatar May 04 '23 00:05 github-actions[bot]