qiniu / kube-ovn / 24703429115

25%

master: 26%

Build Type

Pull #46

github

Committed by zbb88888

Commit Message

fix(controller): promote bgpVipIndexName to package-level const and correct comment

- Move bgpVipIndexName from function-scoped (Run) to package-level const so both
  controller.go (indexer registration) and vip.go (ByIndex call) reference the same
  symbol; a rename is now caught at compile time instead of silently returning empty
  results at runtime.
- Correct misleading comment in enqueueUpdateVirtualIP: the Kubernetes informer does
  call UpdateFunc for status-only updates, so the handler fires on any VIP modification.
  The spec-change block is independent and only enqueues to updateVirtualIPQueue on
  Spec field changes; the bgp_lb_vip re-enqueue block runs on the first IP allocation
  event (oldVip.Status.V4ip == "" → non-empty) regardless of spec changes.

Pull Request Pull Request #46: support BGP EIP management for LoadBalancer（仅 review）

Coverage Stats

168 of 357 new or added lines in 7 files covered. (47.06%)

5 existing lines in 3 files now uncovered.

14046 of 56701 relevant lines covered (24.77%)

0.29 hits per line