From a9e838f2e9823947bb9a5b7c5fccd9b359b4a799 Mon Sep 17 00:00:00 2001 From: Jan Safranek Date: Wed, 10 Jul 2019 15:27:00 +0200 Subject: [PATCH] Remove redundant provisioning tests --- test/e2e/storage/testsuites/provisioning.go | 33 --------------------- 1 file changed, 33 deletions(-) diff --git a/test/e2e/storage/testsuites/provisioning.go b/test/e2e/storage/testsuites/provisioning.go index 8fc3a806e01..21ca668fe61 100644 --- a/test/e2e/storage/testsuites/provisioning.go +++ b/test/e2e/storage/testsuites/provisioning.go @@ -151,16 +151,6 @@ func (p *provisioningTestSuite) defineTests(driver TestDriver, pattern testpatte validateMigrationVolumeOpCounts(f.ClientSet, dInfo.InTreePluginName, l.intreeOps, l.migratedOps) } - ginkgo.It("should provision storage with defaults", func() { - init() - defer cleanup() - - l.testCase.PvCheck = func(claim *v1.PersistentVolumeClaim) { - PVWriteReadSingleNodeCheck(l.cs, claim, framework.NodeSelection{Name: l.config.ClientNodeName}) - } - l.testCase.TestDynamicProvisioning() - }) - ginkgo.It("should provision storage with mount options", func() { if dInfo.SupportedMountOption == nil { framework.Skipf("Driver %q does not define supported mount option - skipping", dInfo.Name) @@ -176,29 +166,6 @@ func (p *provisioningTestSuite) defineTests(driver TestDriver, pattern testpatte l.testCase.TestDynamicProvisioning() }) - ginkgo.It("should access volume from different nodes", func() { - init() - defer cleanup() - - // The assumption is that if the test hasn't been - // locked onto a single node, then the driver is - // usable on all of them *and* supports accessing a volume - // from any node. - if l.config.ClientNodeName != "" { - framework.Skipf("Driver %q only supports testing on one node - skipping", dInfo.Name) - } - - // Ensure that we actually have more than one node. - nodes := framework.GetReadySchedulableNodesOrDie(l.cs) - if len(nodes.Items) <= 1 { - framework.Skipf("need more than one node - skipping") - } - l.testCase.PvCheck = func(claim *v1.PersistentVolumeClaim) { - PVMultiNodeCheck(l.cs, claim, framework.NodeSelection{Name: l.config.ClientNodeName}) - } - l.testCase.TestDynamicProvisioning() - }) - ginkgo.It("should provision storage with snapshot data source [Feature:VolumeSnapshotDataSource]", func() { if !dInfo.Capabilities[CapDataSource] { framework.Skipf("Driver %q does not support populate data from snapshot - skipping", dInfo.Name)