k8snetworkplumbingwg / sriov-cni

DPDK & SR-IOV CNI plugin
Apache License 2.0
309 stars 146 forks source link

Support moveing VF which is eth0 on the host to the namespace #40

Closed Mmduh-483 closed 5 years ago

Mmduh-483 commented 5 years ago

When using multus pluing the first interface will be eth0 and the others will be netX. We need to make sure that before we move VF which is eth0 to the container namespace it should first be renamed to temp name before moving to the namespace. This is to avoid conflict with the eth0 interface which already exist in the namespace.

zshi-redhat commented 5 years ago

If if0name is provided, it's checked to ensure the given interface name is not in the list of "eth0, eth1, lo"; but the if0name is configured after VF be moved into pod namespace, so there is chance that the original VF name conflicts with the default network interface name when moving to pod namespace.

moshe010 commented 5 years ago

So we have fix for that I will request Mmduh-483 to upload it