use different rg for vnet in custom vnet e2e test

kubernetes-sigs · Oct 26, 2023 · c44cd84 · c44cd84
1 parent 6987809
commit c44cd84
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 5 deletions.
diff --git a/azure/scope/cluster.go b/azure/scope/cluster.go
@@ -310,7 +310,7 @@ func (s *ClusterScope) RouteTableSpecs() []azure.ResourceSpecGetter {
 			specs = append(specs, &routetables.RouteTableSpec{
 				Name:           subnet.RouteTable.Name,
 				Location:       s.Location(),
-				ResourceGroup:  s.ResourceGroup(),
+				ResourceGroup:  s.Vnet().ResourceGroup,
 				ClusterName:    s.ClusterName(),
 				AdditionalTags: s.AdditionalTags(),
 			})
@@ -358,7 +358,7 @@ func (s *ClusterScope) NSGSpecs() []azure.ResourceSpecGetter {
 		nsgspecs[i] = &securitygroups.NSGSpec{
 			Name:                     subnet.SecurityGroup.Name,
 			SecurityRules:            subnet.SecurityGroup.SecurityRules,
-			ResourceGroup:            s.ResourceGroup(),
+			ResourceGroup:            s.Vnet().ResourceGroup,
 			Location:                 s.Location(),
 			ClusterName:              s.ClusterName(),
 			AdditionalTags:           s.AdditionalTags(),

diff --git a/templates/test/ci/cluster-template-prow-custom-vnet.yaml b/templates/test/ci/cluster-template-prow-custom-vnet.yaml
diff --git a/templates/test/ci/prow-custom-vnet/patches/custom-vnet.yaml b/templates/test/ci/prow-custom-vnet/patches/custom-vnet.yaml
@@ -5,7 +5,7 @@ metadata:
 spec:
   networkSpec:
     vnet:
-      resourceGroup: ${AZURE_RESOURCE_GROUP}
+      resourceGroup: ${AZURE_CUSTOM_VNET_RESOURCE_GROUP}
       name: ${AZURE_CUSTOM_VNET_NAME}
     subnets:
       - name: ${AZURE_CUSTOM_VNET_NAME}-controlplane-subnet

diff --git a/test/e2e/azure_privatecluster.go b/test/e2e/azure_privatecluster.go
@@ -228,7 +228,7 @@ func SetupExistingVNet(ctx context.Context, vnetCidr string, cpSubnetCidrs, node
 	routetableClient.Authorizer = authorizer
 
 	By("creating a resource group")
-	groupName := os.Getenv(AzureResourceGroup)
+	groupName := os.Getenv(AzureCustomVnetResourceGroup)
 	_, err = groupClient.CreateOrUpdate(ctx, groupName, resources.Group{
 		Location: ptr.To(os.Getenv(AzureLocation)),
 		Tags: map[string]*string{

diff --git a/test/e2e/azure_test.go b/test/e2e/azure_test.go
@@ -135,6 +135,7 @@ var _ = Describe("Workload cluster creation", func() {
 		}
 		dumpSpecResourcesAndCleanup(ctx, cleanInput)
 		Expect(os.Unsetenv(AzureResourceGroup)).To(Succeed())
+		Expect(os.Unsetenv(AzureCustomVnetResourceGroup)).To(Succeed())
 		Expect(os.Unsetenv(AzureVNetName)).To(Succeed())
 		Expect(os.Unsetenv(ClusterIdentityName)).To(Succeed())
 		Expect(os.Unsetenv(ClusterIdentityNamespace)).To(Succeed())
@@ -155,6 +156,7 @@ var _ = Describe("Workload cluster creation", func() {
 				clusterName = getClusterName(clusterNamePrefix, "public-custom-vnet")
 				By("Creating a custom virtual network", func() {
 					Expect(os.Setenv(AzureCustomVNetName, "custom-vnet")).To(Succeed())
+					Expect(os.Setenv(AzureCustomVnetResourceGroup, clusterName+"-vnetrg")).To(Succeed())
 					additionalCleanup = SetupExistingVNet(ctx,
 						"10.0.0.0/16",
 						map[string]string{fmt.Sprintf("%s-controlplane-subnet", os.Getenv(AzureCustomVNetName)): "10.0.0.0/24"},

diff --git a/test/e2e/common.go b/test/e2e/common.go
@@ -57,6 +57,7 @@ const (
 	AzureExtendedLocationType       = "AZURE_EXTENDEDLOCATION_TYPE"
 	AzureExtendedLocationName       = "AZURE_EXTENDEDLOCATION_NAME"
 	AzureResourceGroup              = "AZURE_RESOURCE_GROUP"
+	AzureCustomVnetResourceGroup    = "AZURE_CUSTOM_VNET_RESOURCE_GROUP"
 	AzureVNetName                   = "AZURE_VNET_NAME"
 	AzureCustomVNetName             = "AZURE_CUSTOM_VNET_NAME"
 	AzureInternalLBIP               = "AZURE_INTERNAL_LB_IP"