foundation-model-stack / multi-nic-cni

https://foundation-model-stack.github.io/multi-nic-cni/
Apache License 2.0
34 stars 5 forks source link

Test case should not include prev result in cmdDel #72

Closed sunya-ch closed 1 year ago

sunya-ch commented 1 year ago

Describe the bug A clear and concise description of what the bug is.

This is related to why the daemon-test does not cover the case in https://github.com/foundation-model-stack/multi-nic-cni/issues/69.

The previous result which includes the ipConfigs should not be added to config sent to cmdDel function. In current real deployment, the previous result did not parse by multus.

Signed-off-by: Sunyanan Choochotkaew sunyanan.choochotkaew1@ibm.com