nilpointer during clusterctl move

If .status.network is empty (e.g. during clusterctl move) the
reconciliation will fail due a nil pointer.

This commit skips the first reconciliation of
openStackCluster.Spec.APIServerLoadBalancer.AllowedCIDRs after a clusterctl move.

Signed-off-by: Mario Constanti <[email protected]>

Author: bavarianbidi

pkg/cloud/services/loadbalancer/loadbalancer.go

@@ -134,10 +134,13 @@ func (s *Service) ReconcileLoadBalancer(openStackCluster *infrav1.OpenStackClust
 		}
 
 		if allowedCIDRsSupported {
-			if err := s.getOrUpdateAllowedCIDRS(openStackCluster, listener); err != nil {
-				return err
+			// Skip reconciliation if network status is nil (e.g. during clusterctl move)
+			if openStackCluster.Status.Network != nil {
+				if err := s.getOrUpdateAllowedCIDRS(openStackCluster, listener); err != nil {
+					return err
+				}
+				allowedCIDRs = listener.AllowedCIDRs
 			}
-			allowedCIDRs = listener.AllowedCIDRs
 		}
 	}