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

[v1.0.2] CIDR was removed unexpectedly #46

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.

In busy cluster that api server can be overloaded, CIDR CR was removed. This could be caused by the following code as multi-nic cni operator takes that server error as not found error and remove the resource.

https://github.com/foundation-model-stack/multi-nic-cni/blob/97cd06b9eadab948c6d7e1bb5f5e729ed99f2757/controllers/cidr_handler.go#L137-L142

To Reproduce Steps to reproduce the behavior:

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

Additional context Add any other context about the problem here.

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