test: configurable pvc E2E tests (#2620)

test/e2e/config/config.go

@@ -46,6 +46,7 @@ type Config struct {
 	DebugAfterSuite     bool   `envconfig:"DEBUG_AFTERSUITE" default:"false"`
 	BlockSSHPort        bool   `envconfig:"BLOCK_SSH" default:"false"`
 	AddNodePoolInput    string `envconfig:"ADD_NODE_POOL_INPUT" default:""`
+	TestPVC             bool   `envconfig:"TEST_PVC" default:"false"`
 }
 
 // CustomCloudConfig holds configurations for custom clould

test/e2e/kubernetes/kubernetes_test.go

@@ -1255,7 +1255,8 @@ var _ = Describe("Azure Container Cluster using the Kubernetes Orchestrator", fu
 		It("should create a pv by deploying a pod that consumes a pvc", func() {
 			if !util.IsUsingManagedDisks(eng.ExpandedDefinition.Properties.AgentPoolProfiles) {
 				Skip("Skip PV test for clusters using unmanaged disks")
-			} else if !eng.ExpandedDefinition.Properties.HasNonRegularPriorityScaleset() {
+			} else if !eng.ExpandedDefinition.Properties.HasNonRegularPriorityScaleset() &&
+				cfg.TestPVC {
 				By("Creating a persistent volume claim")
 				pvcName := "azure-disk" // should be the same as in pvc-azuredisk.yaml
 				pvc, err := persistentvolumeclaims.CreatePersistentVolumeClaimsFromFile(filepath.Join(WorkloadDir, "pvc-azuredisk.yaml"), pvcName, "default")

test/e2e/test_cluster_configs/everything.json

@@ -2,7 +2,8 @@
 	"env": {
 		"CREATE_VNET": true,
 		"REGION_OPTIONS": "eastus,westeurope,westus2",
-		"GINKGO_SKIP": "should report all nodes in a Ready state"
+		"GINKGO_SKIP": "should report all nodes in a Ready state",
+		"TEST_PVC": true
 	},
 	"options": {
 		"allowedOrchestratorVersions": [