kubeovn / kube-ovn / 26349097361

24%

master: 25%

Build Type

push

github

Committed by oilbeater

Commit Message

fix(controller): avoid panic in external VPC sync when no logical switch is found (#6777)

Three issues addressed in syncExternalVpc / getNonKubeovnRouterStatus:

- Fix index out of range panic at external_vpc.go when ListLogicalSwitch
  returns an empty slice without error: the previous guard only rejected
  err != nil or len > 1, so len == 0 fell through to switches[0] (#6774).
- Move the vpc.Status.Subnets reset to before the append loop so the
  subnets collected from the logical router are actually persisted on
  the freshly created external VPC instead of being overwritten with [].
- Split combined error/length guards in the peerPorts block, fix %s
  formatting of a util.Port struct, and inline the temporary
  aLogicalSwitch variable.

Closes #6774

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
(cherry picked from commit 9aa6246b9)
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

Coverage Stats

0 of 12 new or added lines in 1 file covered. (0.0%)

2 existing lines in 1 file now uncovered.

13151 of 54815 relevant lines covered (23.99%)

0.28 hits per line