Merge pull request #4815 from k8s-infra-cherrypick-robot/cherry-pick-4725-to-release-1.14

k8s-ci-robot · web-flow · commit 03ea504bf933 · 2024-05-07T10:43:58.000-07:00
[release-1.14] Allow upper-case resource names
diff --git a/azure/services/agentpools/spec.go b/azure/services/agentpools/spec.go
@@ -166,13 +166,13 @@ func (s *AgentPoolSpec) ResourceRef() genruntime.MetaObject {
 	if s.Preview {
 		return &asocontainerservicev1preview.ManagedClustersAgentPool{
 			ObjectMeta: metav1.ObjectMeta{
-				Name: s.Name,
+				Name: azure.GetNormalizedKubernetesName(s.Name),
 			},
 		}
 	}
 	return &asocontainerservicev1.ManagedClustersAgentPool{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
diff --git a/azure/services/aksextensions/spec.go b/azure/services/aksextensions/spec.go
@@ -24,6 +24,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/utils/ptr"
 	infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"
+	"sigs.k8s.io/cluster-api-provider-azure/azure"
 )
 
 // AKSExtensionSpec defines the specification for an AKS Extension.
@@ -47,7 +48,7 @@ type AKSExtensionSpec struct {
 func (s *AKSExtensionSpec) ResourceRef() *asokubernetesconfigurationv1.Extension {
 	return &asokubernetesconfigurationv1.Extension{
 		ObjectMeta: metav1.ObjectMeta{
-			Name:      s.Name,
+			Name:      azure.GetNormalizedKubernetesName(s.Name),
 			Namespace: s.Namespace,
 		},
 	}
diff --git a/azure/services/bastionhosts/spec.go b/azure/services/bastionhosts/spec.go
@@ -26,6 +26,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/utils/ptr"
 	infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"
+	"sigs.k8s.io/cluster-api-provider-azure/azure"
 )
 
 // AzureBastionSpec defines the specification for azure bastion feature.
@@ -44,7 +45,7 @@ type AzureBastionSpec struct {
 func (s *AzureBastionSpec) ResourceRef() *asonetworkv1.BastionHost {
 	return &asonetworkv1.BastionHost{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
@@ -60,7 +61,7 @@ func (s *AzureBastionSpec) Parameters(ctx context.Context, existingBastionHost *
 	bastionHost.Spec.AzureName = s.Name
 	bastionHost.Spec.Location = ptr.To(s.Location)
 	bastionHost.Spec.Owner = &genruntime.KnownResourceReference{
-		Name: s.ResourceGroup,
+		Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),
 	}
 	bastionHost.Spec.Tags = infrav1.Build(infrav1.BuildParams{
 		ClusterName: s.ClusterName,
diff --git a/azure/services/groups/groups.go b/azure/services/groups/groups.go
@@ -70,7 +70,7 @@ func (s *Service) IsManaged(ctx context.Context) (bool, error) {
 		// resource. We also need to check that deleting the ASO resource will really
 		// delete the underlying resource group by checking the ASO reconcile-policy.
 		group := spec.ResourceRef()
-		groupName := group.Name
+		groupName := azure.GetNormalizedKubernetesName(group.Name)
 		groupNamespace := s.Scope.ASOOwner().GetNamespace()
 		err = s.Scope.GetClient().Get(ctx, client.ObjectKey{Namespace: groupNamespace, Name: groupName}, group)
 		if err != nil || group.GetAnnotations()[asoannotations.ReconcilePolicy] != string(asoannotations.ReconcilePolicyManage) {
diff --git a/azure/services/groups/spec.go b/azure/services/groups/spec.go
@@ -23,6 +23,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/utils/ptr"
 	infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"
+	"sigs.k8s.io/cluster-api-provider-azure/azure"
 	"sigs.k8s.io/cluster-api-provider-azure/azure/services/aso"
 )
 
@@ -39,7 +40,7 @@ type GroupSpec struct {
 func (s *GroupSpec) ResourceRef() *asoresourcesv1.ResourceGroup {
 	return &asoresourcesv1.ResourceGroup{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
diff --git a/azure/services/managedclusters/spec.go b/azure/services/managedclusters/spec.go
@@ -385,13 +385,13 @@ func (s *ManagedClusterSpec) ResourceRef() genruntime.MetaObject {
 	if s.Preview {
 		return &asocontainerservicev1preview.ManagedCluster{
 			ObjectMeta: metav1.ObjectMeta{
-				Name: s.Name,
+				Name: azure.GetNormalizedKubernetesName(s.Name),
 			},
 		}
 	}
 	return &asocontainerservicev1.ManagedCluster{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
@@ -441,7 +441,7 @@ func (s *ManagedClusterSpec) Parameters(ctx context.Context, existingObj genrunt
 
 	managedCluster.Spec.AzureName = s.Name
 	managedCluster.Spec.Owner = &genruntime.KnownResourceReference{
-		Name: s.ResourceGroup,
+		Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),
 	}
 	managedCluster.Spec.Identity = &asocontainerservicev1.ManagedClusterIdentity{
 		Type: ptr.To(asocontainerservicev1.ManagedClusterIdentity_Type_SystemAssigned),
diff --git a/azure/services/natgateways/spec.go b/azure/services/natgateways/spec.go
@@ -43,7 +43,7 @@ type NatGatewaySpec struct {
 func (s *NatGatewaySpec) ResourceRef() *asonetworkv1.NatGateway {
 	return &asonetworkv1.NatGateway{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
@@ -59,7 +59,7 @@ func (s *NatGatewaySpec) Parameters(ctx context.Context, existingNatGateway *aso
 
 	natGateway.Spec.AzureName = s.Name
 	natGateway.Spec.Owner = &genruntime.KnownResourceReference{
-		Name: s.ResourceGroup,
+		Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),
 	}
 	natGateway.Spec.Location = ptr.To(s.Location)
 	natGateway.Spec.Sku = &asonetworkv1.NatGatewaySku{
diff --git a/azure/services/privateendpoints/spec.go b/azure/services/privateendpoints/spec.go
@@ -25,6 +25,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/utils/ptr"
 	infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"
+	"sigs.k8s.io/cluster-api-provider-azure/azure"
 )
 
 // PrivateLinkServiceConnection defines the specification for a private link service connection associated with a private endpoint.
@@ -54,7 +55,7 @@ type PrivateEndpointSpec struct {
 func (s *PrivateEndpointSpec) ResourceRef() *asonetworkv1.PrivateEndpoint {
 	return &asonetworkv1.PrivateEndpoint{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
@@ -129,7 +130,7 @@ func (s *PrivateEndpointSpec) Parameters(ctx context.Context, existingPrivateEnd
 	}
 
 	privateEndpoint.Spec.Owner = &genruntime.KnownResourceReference{
-		Name: s.ResourceGroup,
+		Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),
 	}
 
 	privateEndpoint.Spec.Subnet = &asonetworkv1.Subnet_PrivateEndpoint_SubResourceEmbedded{
diff --git a/azure/services/privateendpoints/spec_test.go b/azure/services/privateendpoints/spec_test.go
@@ -31,7 +31,7 @@ import (
 var (
 	fakePrivateEndpoint = PrivateEndpointSpec{
 		Name:                       "test_private_endpoint_1",
-		ResourceGroup:              "test_rg",
+		ResourceGroup:              "test-rg",
 		Location:                   "test_location",
 		CustomNetworkInterfaceName: "test_if_name",
 		PrivateIPAddresses:         []string{"1.2.3.4", "5.6.7.8"},
diff --git a/azure/services/subnets/spec.go b/azure/services/subnets/spec.go
@@ -49,7 +49,7 @@ func (s *SubnetSpec) ResourceRef() *asonetworkv1.VirtualNetworksSubnet {
 		ObjectMeta: metav1.ObjectMeta{
 			// s.Name isn't unique per-cluster, so combine with vnet name to avoid collisions.
 			// ToLower makes the name compatible with standard Kubernetes name requirements.
-			Name: s.VNetName + "-" + strings.ToLower(s.Name),
+			Name: azure.GetNormalizedKubernetesName(s.VNetName + "-" + strings.ToLower(s.Name)),
 		},
 	}
 }
@@ -64,7 +64,7 @@ func (s *SubnetSpec) Parameters(ctx context.Context, existing *asonetworkv1.Virt
 	subnet.Spec = asonetworkv1.VirtualNetworks_Subnet_Spec{
 		AzureName: s.Name,
 		Owner: &genruntime.KnownResourceReference{
-			Name: s.VNetName,
+			Name: azure.GetNormalizedKubernetesName(s.VNetName),
 		},
 		AddressPrefixes: s.CIDRs,
 	}
diff --git a/azure/services/virtualnetworks/spec.go b/azure/services/virtualnetworks/spec.go
@@ -24,6 +24,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/utils/ptr"
 	infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"
+	"sigs.k8s.io/cluster-api-provider-azure/azure"
 	"sigs.k8s.io/cluster-api-provider-azure/azure/converters"
 )
 
@@ -42,7 +43,7 @@ type VNetSpec struct {
 func (s *VNetSpec) ResourceRef() *asonetworkv1.VirtualNetwork {
 	return &asonetworkv1.VirtualNetwork{
 		ObjectMeta: metav1.ObjectMeta{
-			Name: s.Name,
+			Name: azure.GetNormalizedKubernetesName(s.Name),
 		},
 	}
 }
@@ -66,7 +67,7 @@ func (s *VNetSpec) Parameters(ctx context.Context, existing *asonetworkv1.Virtua
 
 	vnet.Spec.AzureName = s.Name
 	vnet.Spec.Owner = &genruntime.KnownResourceReference{
-		Name: s.ResourceGroup,
+		Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),
 	}
 	vnet.Spec.Location = ptr.To(s.Location)
 	vnet.Spec.ExtendedLocation = converters.ExtendedLocationToNetworkASO(s.ExtendedLocation)
diff --git a/docs/book/src/topics/managedcluster.md b/docs/book/src/topics/managedcluster.md
@@ -35,6 +35,8 @@ export AZURE_LOCATION="southcentralus"
 export AZURE_RESOURCE_GROUP="${CLUSTER_NAME}"
 ```
 
+***NOTE***: `${CLUSTER_NAME}` should adhere to the RFC 1123 standard. This means that it must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character.
+
 Create a new service principal and save to a local file:
 
 ```bash
diff --git a/test/e2e/aks_marketplace.go b/test/e2e/aks_marketplace.go
@@ -43,7 +43,7 @@ type AKSMarketplaceExtensionSpecInput struct {
 }
 
 const (
-	extensionName         = "aks-marketplace-extension"
+	extensionName         = "AKS-marketplace-extension" // Test that upper case name is allowed
 	officialExtensionName = "official-aks-extension"
 )
 

Original file line number	Diff line number	Diff line change
`@@ -166,13 +166,13 @@ func (s *AgentPoolSpec) ResourceRef() genruntime.MetaObject {`
`166`	`166`	`if s.Preview {`
`167`	`167`	`return &asocontainerservicev1preview.ManagedClustersAgentPool{`
`168`	`168`	`ObjectMeta: metav1.ObjectMeta{`
`169`		`- Name: s.Name,`
	`169`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`170`	`170`	`},`
`171`	`171`	`}`
`172`	`172`	`}`
`173`	`173`	`return &asocontainerservicev1.ManagedClustersAgentPool{`
`174`	`174`	`ObjectMeta: metav1.ObjectMeta{`
`175`		`- Name: s.Name,`
	`175`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`176`	`176`	`},`
`177`	`177`	`}`
`178`	`178`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ import (`
`26`	`26`	`metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"`
`27`	`27`	`"k8s.io/utils/ptr"`
`28`	`28`	`infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"`
	`29`	`+ "sigs.k8s.io/cluster-api-provider-azure/azure"`
`29`	`30`	`)`
`30`	`31`
`31`	`32`	`// AzureBastionSpec defines the specification for azure bastion feature.`
`@@ -44,7 +45,7 @@ type AzureBastionSpec struct {`
`44`	`45`	`func (s AzureBastionSpec) ResourceRef() asonetworkv1.BastionHost {`
`45`	`46`	`return &asonetworkv1.BastionHost{`
`46`	`47`	`ObjectMeta: metav1.ObjectMeta{`
`47`		`- Name: s.Name,`
	`48`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`48`	`49`	`},`
`49`	`50`	`}`
`50`	`51`	`}`
`@@ -60,7 +61,7 @@ func (s AzureBastionSpec) Parameters(ctx context.Context, existingBastionHost `
`60`	`61`	`bastionHost.Spec.AzureName = s.Name`
`61`	`62`	`bastionHost.Spec.Location = ptr.To(s.Location)`
`62`	`63`	`bastionHost.Spec.Owner = &genruntime.KnownResourceReference{`
`63`		`- Name: s.ResourceGroup,`
	`64`	`+ Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),`
`64`	`65`	`}`
`65`	`66`	`bastionHost.Spec.Tags = infrav1.Build(infrav1.BuildParams{`
`66`	`67`	`ClusterName: s.ClusterName,`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,7 @@ import (`
`23`	`23`	`metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"`
`24`	`24`	`"k8s.io/utils/ptr"`
`25`	`25`	`infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"`
	`26`	`+ "sigs.k8s.io/cluster-api-provider-azure/azure"`
`26`	`27`	`"sigs.k8s.io/cluster-api-provider-azure/azure/services/aso"`
`27`	`28`	`)`
`28`	`29`
`@@ -39,7 +40,7 @@ type GroupSpec struct {`
`39`	`40`	`func (s GroupSpec) ResourceRef() asoresourcesv1.ResourceGroup {`
`40`	`41`	`return &asoresourcesv1.ResourceGroup{`
`41`	`42`	`ObjectMeta: metav1.ObjectMeta{`
`42`		`- Name: s.Name,`
	`43`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`43`	`44`	`},`
`44`	`45`	`}`
`45`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -385,13 +385,13 @@ func (s *ManagedClusterSpec) ResourceRef() genruntime.MetaObject {`
`385`	`385`	`if s.Preview {`
`386`	`386`	`return &asocontainerservicev1preview.ManagedCluster{`
`387`	`387`	`ObjectMeta: metav1.ObjectMeta{`
`388`		`- Name: s.Name,`
	`388`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`389`	`389`	`},`
`390`	`390`	`}`
`391`	`391`	`}`
`392`	`392`	`return &asocontainerservicev1.ManagedCluster{`
`393`	`393`	`ObjectMeta: metav1.ObjectMeta{`
`394`		`- Name: s.Name,`
	`394`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`395`	`395`	`},`
`396`	`396`	`}`
`397`	`397`	`}`
`@@ -441,7 +441,7 @@ func (s *ManagedClusterSpec) Parameters(ctx context.Context, existingObj genrunt`
`441`	`441`
`442`	`442`	`managedCluster.Spec.AzureName = s.Name`
`443`	`443`	`managedCluster.Spec.Owner = &genruntime.KnownResourceReference{`
`444`		`- Name: s.ResourceGroup,`
	`444`	`+ Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),`
`445`	`445`	`}`
`446`	`446`	`managedCluster.Spec.Identity = &asocontainerservicev1.ManagedClusterIdentity{`
`447`	`447`	`Type: ptr.To(asocontainerservicev1.ManagedClusterIdentity_Type_SystemAssigned),`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ type NatGatewaySpec struct {`
`43`	`43`	`func (s NatGatewaySpec) ResourceRef() asonetworkv1.NatGateway {`
`44`	`44`	`return &asonetworkv1.NatGateway{`
`45`	`45`	`ObjectMeta: metav1.ObjectMeta{`
`46`		`- Name: s.Name,`
	`46`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`47`	`47`	`},`
`48`	`48`	`}`
`49`	`49`	`}`
`@@ -59,7 +59,7 @@ func (s NatGatewaySpec) Parameters(ctx context.Context, existingNatGateway aso`
`59`	`59`
`60`	`60`	`natGateway.Spec.AzureName = s.Name`
`61`	`61`	`natGateway.Spec.Owner = &genruntime.KnownResourceReference{`
`62`		`- Name: s.ResourceGroup,`
	`62`	`+ Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),`
`63`	`63`	`}`
`64`	`64`	`natGateway.Spec.Location = ptr.To(s.Location)`
`65`	`65`	`natGateway.Spec.Sku = &asonetworkv1.NatGatewaySku{`
Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,7 @@ import (`
`25`	`25`	`metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"`
`26`	`26`	`"k8s.io/utils/ptr"`
`27`	`27`	`infrav1 "sigs.k8s.io/cluster-api-provider-azure/api/v1beta1"`
	`28`	`+ "sigs.k8s.io/cluster-api-provider-azure/azure"`
`28`	`29`	`)`
`29`	`30`
`30`	`31`	`// PrivateLinkServiceConnection defines the specification for a private link service connection associated with a private endpoint.`
`@@ -54,7 +55,7 @@ type PrivateEndpointSpec struct {`
`54`	`55`	`func (s PrivateEndpointSpec) ResourceRef() asonetworkv1.PrivateEndpoint {`
`55`	`56`	`return &asonetworkv1.PrivateEndpoint{`
`56`	`57`	`ObjectMeta: metav1.ObjectMeta{`
`57`		`- Name: s.Name,`
	`58`	`+ Name: azure.GetNormalizedKubernetesName(s.Name),`
`58`	`59`	`},`
`59`	`60`	`}`
`60`	`61`	`}`
`@@ -129,7 +130,7 @@ func (s *PrivateEndpointSpec) Parameters(ctx context.Context, existingPrivateEnd`
`129`	`130`	`}`
`130`	`131`
`131`	`132`	`privateEndpoint.Spec.Owner = &genruntime.KnownResourceReference{`
`132`		`- Name: s.ResourceGroup,`
	`133`	`+ Name: azure.GetNormalizedKubernetesName(s.ResourceGroup),`
`133`	`134`	`}`
`134`	`135`
`135`	`136`	`privateEndpoint.Spec.Subnet = &asonetworkv1.Subnet_PrivateEndpoint_SubResourceEmbedded{`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ func (s SubnetSpec) ResourceRef() asonetworkv1.VirtualNetworksSubnet {`
`49`	`49`	`ObjectMeta: metav1.ObjectMeta{`
`50`	`50`	`// s.Name isn't unique per-cluster, so combine with vnet name to avoid collisions.`
`51`	`51`	`// ToLower makes the name compatible with standard Kubernetes name requirements.`
`52`		`- Name: s.VNetName + "-" + strings.ToLower(s.Name),`
	`52`	`+ Name: azure.GetNormalizedKubernetesName(s.VNetName + "-" + strings.ToLower(s.Name)),`
`53`	`53`	`},`
`54`	`54`	`}`
`55`	`55`	`}`
`@@ -64,7 +64,7 @@ func (s SubnetSpec) Parameters(ctx context.Context, existing asonetworkv1.Virt`
`64`	`64`	`subnet.Spec = asonetworkv1.VirtualNetworks_Subnet_Spec{`
`65`	`65`	`AzureName: s.Name,`
`66`	`66`	`Owner: &genruntime.KnownResourceReference{`
`67`		`- Name: s.VNetName,`
	`67`	`+ Name: azure.GetNormalizedKubernetesName(s.VNetName),`
`68`	`68`	`},`
`69`	`69`	`AddressPrefixes: s.CIDRs,`
`70`	`70`	`}`