kubeovn / kube-ovn / 23381655783

27%

push

github

refactor(controller): eliminate duplicate OVN/K8s queries in pod controller (#6485)

* refactor(controller): merge duplicate ListNormalLogicalSwitchPorts calls in handleDeletePod

Hoist the ListNormalLogicalSwitchPorts query before the VM pod branch
so both the VM migrate-options cleanup and the later port-group/LSP
deletion share a single OVN-NB round trip instead of two identical ones.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

* refactor(controller): remove duplicate nodesLister.Get in reconcileRouteSubnets

The inner nodesLister.Get(pod.Spec.NodeName) at L820 is redundant since
the same node was already fetched at L750 and the result is still valid.
The inner variable was shadowing the outer one without being used.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

---------

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

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

12945 of 55206 relevant lines covered (23.45%)

0.27 hits per line