From b55a9e7a6bcf14e57dd79351a9ac114a5f8e0478 Mon Sep 17 00:00:00 2001 From: Matt Pryor Date: Fri, 29 Apr 2022 13:46:44 +0100 Subject: [PATCH] Fix nil pointer reference --- controllers/openstackcluster_controller.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/controllers/openstackcluster_controller.go b/controllers/openstackcluster_controller.go index 25d532d39d..a71f82dae7 100644 --- a/controllers/openstackcluster_controller.go +++ b/controllers/openstackcluster_controller.go @@ -221,12 +221,12 @@ func deleteBastion(scope *scope.Scope, cluster *clusterv1.Cluster, openStackClus } } } - } - instanceSpec := bastionToInstanceSpec(openStackCluster, cluster.Name) - if err = computeService.DeleteInstance(openStackCluster, instanceSpec, instanceStatus); err != nil { - handleUpdateOSCError(openStackCluster, errors.Errorf("failed to delete bastion: %v", err)) - return errors.Errorf("failed to delete bastion: %v", err) + instanceSpec := bastionToInstanceSpec(openStackCluster, cluster.Name) + if err = computeService.DeleteInstance(openStackCluster, instanceSpec, instanceStatus); err != nil { + handleUpdateOSCError(openStackCluster, errors.Errorf("failed to delete bastion: %v", err)) + return errors.Errorf("failed to delete bastion: %v", err) + } } openStackCluster.Status.Bastion = nil