From 079a4ea30c5bffdc59317904771f03a4a3f91e6e Mon Sep 17 00:00:00 2001 From: Jing Xu Date: Tue, 17 Nov 2020 09:53:50 -0800 Subject: [PATCH] Mark some storage tests as LinuxOnly Some storage tests has commands not available in Windows. Mark them as LinuxOnly now. Will check later to see whether equivalent windows commands are available. Change-Id: I41b5668c855b2754a2e332cff4e90ebf2981aca0 --- test/e2e/storage/testsuites/multivolume.go | 8 ++++---- test/e2e/storage/testsuites/volumemode.go | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/test/e2e/storage/testsuites/multivolume.go b/test/e2e/storage/testsuites/multivolume.go index 9c225b515b5..ee0e0cec2b4 100644 --- a/test/e2e/storage/testsuites/multivolume.go +++ b/test/e2e/storage/testsuites/multivolume.go @@ -127,7 +127,7 @@ func (t *multiVolumeTestSuite) DefineTests(driver TestDriver, pattern testpatter // [ node1 ] ==> [ node1 ] // / \ <- same volume mode / \ // [volume1] [volume2] [volume1] [volume2] - ginkgo.It("should access to two volumes with the same volume mode and retain data across pod recreation on the same node", func() { + ginkgo.It("should access to two volumes with the same volume mode and retain data across pod recreation on the same node [LinuxOnly]", func() { // Currently, multiple volumes are not generally available for pre-provisoined volume, // because containerized storage servers, such as iSCSI and rbd, are just returning // a static volume inside container, not actually creating a new volume per request. @@ -157,7 +157,7 @@ func (t *multiVolumeTestSuite) DefineTests(driver TestDriver, pattern testpatter // [ node1 ] ==> [ node2 ] // / \ <- same volume mode / \ // [volume1] [volume2] [volume1] [volume2] - ginkgo.It("should access to two volumes with the same volume mode and retain data across pod recreation on different node", func() { + ginkgo.It("should access to two volumes with the same volume mode and retain data across pod recreation on different node [LinuxOnly]", func() { // Currently, multiple volumes are not generally available for pre-provisoined volume, // because containerized storage servers, such as iSCSI and rbd, are just returning // a static volume inside container, not actually creating a new volume per request. @@ -207,7 +207,7 @@ func (t *multiVolumeTestSuite) DefineTests(driver TestDriver, pattern testpatter // [ node1 ] ==> [ node1 ] // / \ <- different volume mode / \ // [volume1] [volume2] [volume1] [volume2] - ginkgo.It("should access to two volumes with different volume mode and retain data across pod recreation on the same node", func() { + ginkgo.It("should access to two volumes with different volume mode and retain data across pod recreation on the same node [LinuxOnly]", func() { if pattern.VolMode == v1.PersistentVolumeFilesystem { e2eskipper.Skipf("Filesystem volume case should be covered by block volume case -- skipping") } @@ -246,7 +246,7 @@ func (t *multiVolumeTestSuite) DefineTests(driver TestDriver, pattern testpatter // [ node1 ] ==> [ node2 ] // / \ <- different volume mode / \ // [volume1] [volume2] [volume1] [volume2] - ginkgo.It("should access to two volumes with different volume mode and retain data across pod recreation on different node", func() { + ginkgo.It("should access to two volumes with different volume mode and retain data across pod recreation on different node [LinuxOnly]", func() { if pattern.VolMode == v1.PersistentVolumeFilesystem { e2eskipper.Skipf("Filesystem volume case should be covered by block volume case -- skipping") } diff --git a/test/e2e/storage/testsuites/volumemode.go b/test/e2e/storage/testsuites/volumemode.go index 7e962b983f4..1343335968b 100644 --- a/test/e2e/storage/testsuites/volumemode.go +++ b/test/e2e/storage/testsuites/volumemode.go @@ -251,7 +251,7 @@ func (t *volumeModeTestSuite) DefineTests(driver TestDriver, pattern testpattern case testpatterns.DynamicPV: if pattern.VolMode == v1.PersistentVolumeBlock && !isBlockSupported { - ginkgo.It("should fail in binding dynamic provisioned PV to PVC [Slow]", func() { + ginkgo.It("should fail in binding dynamic provisioned PV to PVC [Slow][LinuxOnly]", func() { manualInit() defer cleanup() @@ -344,7 +344,7 @@ func (t *volumeModeTestSuite) DefineTests(driver TestDriver, pattern testpattern framework.ExpectEqual(p.Status.Phase, v1.PodPending, "Pod phase isn't pending") }) - ginkgo.It("should not mount / map unused volumes in a pod", func() { + ginkgo.It("should not mount / map unused volumes in a pod [LinuxOnly]", func() { if pattern.VolMode == v1.PersistentVolumeBlock { skipTestIfBlockNotSupported(driver) }