Skip to content

Commit b0c9bd8

Browse files
committed
Fix delete pool lead to panic
1) seems we don't need port 22, have no idea why use here 2) when pool not exist, should check and ignore following actions
1 parent d43c67f commit b0c9bd8

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

pkg/cloud/openstack/services/networking/loadbalancer.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,14 +274,18 @@ func (s *Service) DeleteLoadBalancerMember(clusterName string, machine *clusterv
274274

275275
lbID := clusterProviderStatus.Network.APIServerLoadBalancer.ID
276276

277-
for _, port := range []int{22, 6443} {
277+
for _, port := range []int{6443} {
278278
lbPortObjectsName := fmt.Sprintf("%s-%d", loadBalancerName, port)
279279
name := lbPortObjectsName + "-" + machine.Name
280280

281281
pool, err := checkIfPoolExists(s.client, lbPortObjectsName)
282282
if err != nil {
283283
return err
284284
}
285+
if pool == nil {
286+
klog.Infof("Pool %s does not exist", lbPortObjectsName)
287+
continue
288+
}
285289

286290
lbMember, err := checkIfLbMemberExists(s.client, pool.ID, name)
287291
if err != nil {

0 commit comments

Comments
 (0)