• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

kubeovn / kube-ovn / 23405076297
27%

Build:
DEFAULT BRANCH: master
Ran 22 Mar 2026 02:26PM UTC
Jobs 1
Files 205
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

22 Mar 2026 02:23PM UTC coverage: 23.695% (+0.07%) from 23.625%
23405076297

push

github

web-flow
fix(controller): prevent dual-stack cross-deletion of local DNS cache policy routes (#6497)

* fix(controller): use address-family filter in addPolicyRouteForLocalDNSCacheOnNode to prevent dual-stack cross-deletion

In dual-stack environments, addPolicyRouteForLocalDNSCacheOnNode used
GetLogicalRouterPoliciesByExtID which only filtered by "node" key,
returning policies from ALL address families. This caused each af call
to delete the other af's policies — e.g., the af=4 call would delete
af=6 policies and vice versa, resulting in only the last-processed
address family having working local DNS cache routes.

Replace GetLogicalRouterPoliciesByExtID with ListLogicalRouterPolicies
using the full externalIDs map (which includes "address-family"), making
it consistent with deletePolicyRouteForLocalDNSCacheOnNode. Also remove
the now-redundant manual ExternalIDs filter in the loop since
ListLogicalRouterPolicies already handles this filtering.

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(test): remove brittle line-number references from test comments

Address Copilot review feedback: replace hard-coded line number
references (L1096, L1102-1116, node.go:1087) with function/behavior
descriptions that won't drift as the file changes.

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>

1 of 2 new or added lines in 1 file covered. (50.0%)

13070 of 55159 relevant lines covered (23.7%)

0.28 hits per line

Uncovered Changes

Lines Coverage ∆ File
1
6.24
4.24% pkg/controller/node.go
Jobs
ID Job ID Ran Files Coverage
1 23405076297.1 22 Mar 2026 02:26PM UTC 205
23.7
GitHub Action Run
Source Files on build 23405076297
  • Tree
  • List 205
  • Changed 1
  • Source Changed 0
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • f1a627c7 on github
  • Prev Build on master (#23404772237)
  • Next Build on master (#23406077511)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc