Merge pull request #743 from syseleven/openstack_dont_delete_lb_on_er…

…rors

Do not Delete LB in Case of Security Group Reconciliation Errors

pkg/cloudprovider/providers/openstack/openstack_loadbalancer.go

@@ -1336,9 +1336,7 @@ func (lbaas *LbaasV2) EnsureLoadBalancer(ctx context.Context, clusterName string
 	if lbaas.opts.ManageSecurityGroups {
 		err := lbaas.ensureSecurityGroup(clusterName, apiService, nodes, loadbalancer)
 		if err != nil {
-			// cleanup what was created so far
-			_ = lbaas.EnsureLoadBalancerDeleted(ctx, clusterName, apiService)
-			return status, err
+			return status, fmt.Errorf("Error reconciling security groups for LB service %v/%v: %v", apiService.Namespace, apiService.Name, err)
 		}
 	}