Merge pull request #3539 from k8s-infra-cherrypick-robot/cherry-pick-3517-to-release-1.8

[release-1.8] use default backend pool name in capz controller if empty

Author: k8s-ci-robot

api/v1beta1/azurecluster_default.go

@@ -193,10 +193,7 @@ func (c *AzureCluster) setAPIServerLBDefaults() {
 			}
 		}
 	}
-
-	if lb.BackendPool.Name == "" {
-		lb.BackendPool.Name = generateBackendAddressPoolName(lb.Name)
-	}
+	c.SetAPIServerLBBackendPoolNameDefault()
 }
 
 // SetNodeOutboundLBDefaults sets the default values for the NodeOutboundLB.
@@ -236,10 +233,7 @@ func (c *AzureCluster) SetNodeOutboundLBDefaults() {
 	}
 
 	c.setOutboundLBFrontendIPs(lb, generateNodeOutboundIPName)
-
-	if lb.BackendPool.Name == "" {
-		lb.BackendPool.Name = generateOutboundBackendAddressPoolName(lb.Name)
-	}
+	c.SetNodeOutboundLBBackendPoolNameDefault()
 }
 
 // SetControlPlaneOutboundLBDefaults sets the default values for the control plane's outbound LB.
@@ -258,9 +252,37 @@ func (c *AzureCluster) SetControlPlaneOutboundLBDefaults() {
 		lb.FrontendIPsCount = pointer.Int32(1)
 	}
 	c.setOutboundLBFrontendIPs(lb, generateControlPlaneOutboundIPName)
+	c.SetControlPlaneOutboundLBBackendPoolNameDefault()
+}
+
+// SetBackendPoolNameDefault defaults the backend pool name of the LBs.
+func (c *AzureCluster) SetBackendPoolNameDefault() {
+	c.SetAPIServerLBBackendPoolNameDefault()
+	c.SetNodeOutboundLBBackendPoolNameDefault()
+	c.SetControlPlaneOutboundLBBackendPoolNameDefault()
+}
+
+// SetAPIServerLBBackendPoolNameDefault defaults the name of the backend pool for apiserver LB.
+func (c *AzureCluster) SetAPIServerLBBackendPoolNameDefault() {
+	apiServerLB := &c.Spec.NetworkSpec.APIServerLB
+	if apiServerLB.BackendPool.Name == "" {
+		apiServerLB.BackendPool.Name = generateBackendAddressPoolName(apiServerLB.Name)
+	}
+}
+
+// SetNodeOutboundLBBackendPoolNameDefault defaults the name of the backend pool for node outbound LB.
+func (c *AzureCluster) SetNodeOutboundLBBackendPoolNameDefault() {
+	nodeOutboundLB := c.Spec.NetworkSpec.NodeOutboundLB
+	if nodeOutboundLB != nil && nodeOutboundLB.BackendPool.Name == "" {
+		nodeOutboundLB.BackendPool.Name = generateOutboundBackendAddressPoolName(nodeOutboundLB.Name)
+	}
+}
 
-	if lb.BackendPool.Name == "" {
-		lb.BackendPool.Name = generateOutboundBackendAddressPoolName(generateControlPlaneOutboundLBName(c.ObjectMeta.Name))
+// SetControlPlaneOutboundLBBackendPoolNameDefault defaults the name of the backend pool for control plane outbound LB.
+func (c *AzureCluster) SetControlPlaneOutboundLBBackendPoolNameDefault() {
+	controlPlaneOutboundLB := c.Spec.NetworkSpec.ControlPlaneOutboundLB
+	if controlPlaneOutboundLB != nil && controlPlaneOutboundLB.BackendPool.Name == "" {
+		controlPlaneOutboundLB.BackendPool.Name = generateOutboundBackendAddressPoolName(generateControlPlaneOutboundLBName(c.ObjectMeta.Name))
 	}
 }
 

controllers/azurecluster_reconciler.go

@@ -85,6 +85,7 @@ func (s *azureClusterService) Reconcile(ctx context.Context) error {
 		return errors.Wrap(err, "failed to get availability zones")
 	}
 
+	s.scope.AzureCluster.SetBackendPoolNameDefault()
 	s.scope.SetDNSName()
 	s.scope.SetControlPlaneSecurityRules()