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

kubeovn / kube-ovn / 24830126592
25%

Build:
DEFAULT BRANCH: master
Ran 23 Apr 2026 10:29AM UTC
Jobs 1
Files 207
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

23 Apr 2026 10:25AM UTC coverage: 24.853% (+0.04%) from 24.817%
24830126592

push

github

web-flow
fix(controller): keep VM LSP port-group memberships when a sibling pod is alive (#6666)

* fix(controller): keep VM LSP port-group memberships when a sibling pod is alive

For VM pods with EnableKeepVMIP=true a single LSP is shared across every
virt-launcher pod of the VM (ExternalIDs["pod"] is keyed by VM name, not
by pod name). When kubernetes GCs a completed virt-launcher pod — most
commonly the source of a successful live migration — handleDeletePod
finds that LSP, and because isVMToDel is false it calls
RemovePortFromPortGroups(port.Name) with no port-group names, which
strips the LSP from every port group (node, subnet, security group,
network policy).

The LSP is still in use by the currently running destination pod, so the
active VM pod loses its memberships and connectivity until
kube-ovn-controller is restarted and all pods are re-synced.

Before stripping port-group memberships in the keepIPCR branch, check
whether another alive virt-launcher pod exists for the same VMI. If so,
the memberships belong to that sibling and must be preserved.

Fixes #6665

Signed-off-by: Andrei Kvapil <andrei.kvapil@aenix.io>

* fix(controller): rewrite vm lsp cleanup if-else chain as switch

Satisfies gocritic's ifElseChain rule; no behavior change.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

---------

Signed-off-by: Andrei Kvapil <andrei.kvapil@aenix.io>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-authored-by: Mengxin Liu <liumengxinfly@gmail.com>

11 of 29 new or added lines in 1 file covered. (37.93%)

2 existing lines in 1 file now uncovered.

14067 of 56601 relevant lines covered (24.85%)

0.29 hits per line

Uncovered Changes

Lines Coverage ∆ File
18
15.78
1.08% pkg/controller/pod.go

Coverage Regressions

Lines Coverage ∆ File
2
61.61
-0.62% pkg/ovs/ovn-nb-bfd.go
Jobs
ID Job ID Ran Files Coverage
1 24830126592.1 23 Apr 2026 10:29AM UTC 207
24.85
GitHub Action Run
Source Files on build 24830126592
  • Tree
  • List 207
  • Changed 2
  • Source Changed 0
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • 25878a13 on github
  • Prev Build on master (#24823959467)
  • Next Build on master (#24877779623)
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