kubeovn / kube-ovn / 27870602907

25%

master: 27%

push

github

fix(ovs): make DeleteNat idempotent when target NAT row is absent (#6911)

DeleteNat used to return an error ("not found logical router ... nat ...")
when GetNat returned no row. This caused problems for callers such as
the OvnFip delete handler, which could then fail to clear its finalizer
and requeue forever.

Change GetNat to ignore-not-found and return success (nil) if the NAT
row is already gone. This matches the pattern used elsewhere and lets
reconciles proceed cleanly.

A regression test is added to verify both the never-existed case and
double-delete after create.

Signed-off-by: Apinant U-suwantim <Hello@Apinant.dev>
(cherry picked from commit b1cccbd38)

7 of 7 new or added lines in 1 file covered. (100.0%)

13438 of 54949 relevant lines covered (24.46%)

0.28 hits per line