kubeovn / kube-ovn / 23729580444

26%

Build Type

push

github

Committed by web-flow

Commit Message

fix(daemon): replace bulk netlink dumps with targeted queries (#6540)

* fix(daemon): replace bulk netlink dumps with targeted queries

Replace unnecessary netlink full-table dumps with targeted queries to
reduce NLM_F_DUMP_INTR risk on busy nodes:

- removeDefaultRoute: use RouteListFiltered with RT_FILTER_DST to query
  only default routes for the needed address family, instead of dumping
  all routes across all families
- getNicExistRoutes: deduplicate by address family to avoid redundant
  RouteList calls when gateway contains multiple entries of the same
  protocol
- FindKubeOVNAutoCreatedInterfaces: use link.Attrs().Alias (parsed from
  IFLA_IFALIAS by netlink) instead of per-interface sysfs reads, and
  remove the now-unused GetInterfaceAlias/IsKubeOVNAutoCreatedInterface
  helpers

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>

* fix(daemon): revert getNicExistRoutes change as it has no real benefit

The gateway string in practice is either a single address or one V4 + one
V6 address, so the original loop already makes at most one RouteList call
per family. The deduplication adds code complexity for a scenario that
does not occur.

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>

* chore: remove unnecessary comment in DetectVlanInterfaces

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>

Coverage Stats

0 of 25 new or added lines in 2 files covered. (0.0%)

1 existing line in 1 file now uncovered.

13534 of 55687 relevant lines covered (24.3%)

0.28 hits per line