kubeovn / kube-ovn / 26336548260

25%

Build Type

push

github

Committed by web-flow

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>

Coverage Stats

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

2 existing lines in 1 file now uncovered.

14538 of 57473 relevant lines covered (25.3%)

0.29 hits per line