cybozu-go / coil

CNI plugin for Kubernetes designed for scalability and extensibility
Apache License 2.0
158 stars 18 forks source link

[Bug] Coild is not updating `podCIDR` and `podCIDRs` in the node objects. #194

Closed Cellebyte closed 2 years ago

Cellebyte commented 2 years ago

Describe the bug

When I describe node objects I expect correct podCIDR to have easier debugging capabilities.

Environments

To Reproduce Steps to reproduce the behavior:

  1. Go to kubectl get nodes
  2. Describe a node object
  3. Scroll down to podCIDR podCIDRs
  4. See non updated CIDR notations

Expected behavior If coild assigns podCIDR it should also update the node object accordingly.

Additional context

kubectl describe node
PodCIDR:                      172.30.8.0/24
PodCIDRs:                     172.30.8.0/24

kubectl get addressblocks
default-4     <nodename>                   default   172.30.1.0/26 
ymmt2005 commented 2 years ago

It's not a bug.

podCIDR is meant for simple network plugins such as Flannel. cf. https://github.com/kubernetes/kubernetes/issues/57130

A more featureful network plugin like Coil assigns multiple CIDRs for a Node, so podCIDR is simply useless. (note that podCIDRs is meant for IPv4/IPv6 dual stack network.)