From 2298dc612c0b814fefc4c64093697b3fb63919a1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?L=C3=A9iy=C3=AC=20Zhang?= Date: Thu, 11 Nov 2021 23:32:44 +0000 Subject: [PATCH] skip in-tree and default Dynamic Provisioner test when there's no default storage class --- test/e2e/storage/volume_provisioning.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/e2e/storage/volume_provisioning.go b/test/e2e/storage/volume_provisioning.go index edc8249dfae..4d071fc3356 100644 --- a/test/e2e/storage/volume_provisioning.go +++ b/test/e2e/storage/volume_provisioning.go @@ -147,7 +147,7 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() { timeouts = f.Timeouts }) - ginkgo.Describe("DynamicProvisioner [Slow]", func() { + ginkgo.Describe("DynamicProvisioner [Slow] [Feature:StorageProvider]", func() { ginkgo.It("should provision storage with different parameters", func() { // This test checks that dynamic provisioning can provision a volume @@ -604,6 +604,7 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() { ginkgo.Describe("DynamicProvisioner Default", func() { ginkgo.It("should create and delete default persistent volumes [Slow]", func() { e2eskipper.SkipUnlessProviderIs("openstack", "gce", "aws", "gke", "vsphere", "azure") + e2epv.SkipIfNoDefaultStorageClass(c) ginkgo.By("creating a claim with no annotation") test := testsuites.StorageClassTest{ @@ -629,6 +630,7 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() { // Modifying the default storage class can be disruptive to other tests that depend on it ginkgo.It("should be disabled by changing the default annotation [Serial] [Disruptive]", func() { e2eskipper.SkipUnlessProviderIs("openstack", "gce", "aws", "gke", "vsphere", "azure") + e2epv.SkipIfNoDefaultStorageClass(c) scName, scErr := e2epv.GetDefaultStorageClassName(c) framework.ExpectNoError(scErr)