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

kubeovn / kube-ovn / 23180651498
23%
master: 23%

Build:
Build:
LAST BUILD BRANCH: renovate/master-kubevirt.io-client-go-1.x
DEFAULT BRANCH: master
Ran 17 Mar 2026 05:59AM 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

17 Mar 2026 05:56AM UTC coverage: 23.472% (+0.02%) from 23.456%
23180651498

Pull #6461

github

oilbeater
fix(controller): remove Service ClusterIP from NetworkPolicy address sets

NetworkPolicy egress ACLs use OVN's apply-after-lb=true option, which
means they execute after LB DNAT. At that stage, the destination IP is
already the Pod IP (not the Service ClusterIP). Adding Service ClusterIPs
to the egress allow address set is therefore unnecessary.

Worse, it causes a race condition: when LB logical flows are not yet
compiled by northd, ovn-trace sees the un-DNAT'd Service ClusterIP at
the after-lb ACL stage. Since the Service ClusterIP is in the allow
address set, the packet is incorrectly allowed through and routed to
the node instead of the target pod. This manifests as flaky e2e test
failures in "kubectl ko trace ... should work with network policy".

Remove the Service ClusterIP logic from fetchPolicySelectedAddresses()
and clean up the now-unused helper functions: shouldIncludeServiceIPs,
svcMatchPods, getProtocolSvcIP, and isSvcMatchPod.

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>
Pull Request #6461: fix(controller): remove Service ClusterIP from NetworkPolicy address sets

12887 of 54903 relevant lines covered (23.47%)

0.27 hits per line

Jobs
ID Job ID Ran Files Coverage
1 23180651498.1 17 Mar 2026 05:59AM UTC 205
23.47
GitHub Action Run
Source Files on build 23180651498
  • Tree
  • List 205
  • Changed 1
  • Source Changed 0
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Pull Request #6461
  • PR Base - master (#23175537884)
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