Re-enable write-read pv check in volume provisioning tests

test/e2e/storage/csi_volumes.go

@@ -132,7 +132,6 @@ func (h *hostpathCSIDriver) createStorageClassTest(node v1.Node) storageClassTes
 		claimSize:    "1Gi",
 		expectedSize: "1Gi",
 		nodeName:     node.Name,
-		attach:       true,
 	}
 }
 
@@ -198,7 +197,6 @@ func (g *gcePDCSIDriver) createStorageClassTest(node v1.Node) storageClassTest {
 		claimSize:    "5Gi",
 		expectedSize: "5Gi",
 		nodeName:     node.Name,
-		attach:       true,
 	}
 }
 

test/e2e/storage/volume_provisioning.go

@@ -56,7 +56,7 @@ type storageClassTest struct {
 	expectedSize       string
 	pvCheck            func(volume *v1.PersistentVolume) error
 	nodeName           string
-	attach         bool
+	skipWriteReadCheck bool
 	volumeMode         *v1.PersistentVolumeMode
 }
 
@@ -132,7 +132,7 @@ func testDynamicProvisioning(t storageClassTest, client clientset.Interface, cla
 		Expect(err).NotTo(HaveOccurred())
 	}
 
-	if t.attach {
+	if !t.skipWriteReadCheck {
 		// We start two pods:
 		// - The first writes 'hello word' to the /mnt/test (= the volume).
 		// - The second one runs grep 'hello world' on /mnt/test.
@@ -801,7 +801,7 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() {
 				claimSize:          "2Gi",
 				expectedSize:       "2Gi",
 				parameters:         map[string]string{"resturl": serverUrl},
-				attach:       false,
+				skipWriteReadCheck: true,
 			}
 
 			// GCE/GKE
@@ -837,6 +837,7 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() {
 				claimSize:          "2Gi",
 				expectedSize:       "2Gi",
 				volumeMode:         &block,
+				skipWriteReadCheck: true,
 			}
 			// gce or gke
 			if getDefaultPluginName() == "kubernetes.io/gce-pd" {