kubeovn / kube-ovn / 27260162393

25%

master: 27%

Build Type

push

github

Committed by oilbeater

Commit Message

fix(controller): avoid nil pointer panic when attachment default subnet is gone (#6843)

getPodAttachmentNet falls back to getPodDefaultSubnet when an OVN attachment
network resolves to no subnet. getPodDefaultSubnet returns (nil, nil) for a
terminating pod whose top-level logical_switch annotation points at an
already-deleted subnet, leaving the orphaned ip cr to gc. The caller only
checked the error and then dereferenced subnet.Name, panicking the controller.

Skip the unresolvable attachment when the default subnet is nil, matching the
behavior of the other not-found branches in the same function.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
(cherry picked from commit 1fa2b218d)

Coverage Stats

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

54 existing lines in 1 file now uncovered.

14309 of 56764 relevant lines covered (25.21%)

0.29 hits per line