Merge pull request kubernetes#39547 from copejon/fix-pv-e2e-flake

Automatic merge from submit-queue (batch tested with PRs 39495, 39547)

Tag persistent volume PersistentVolume E2E [Volume][Serial][Flaky]

**What this PR does / why we need it**:
When run parallel with other tests that use PV(C)s, cross-test binding causes flakes.  Add `[Serial]` tag.
**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: f
Partly addresses kubernetes#39119 
 
**Special notes for your reviewer**:
cc @saad-ali @jsafrane @jeffvance

test/e2e/persistent_volumes.go

@@ -560,31 +560,31 @@ var _ = framework.KubeDescribe("PersistentVolumes", func() {
 			// Create an nfs PV, then a claim that matches the PV, and a pod that
 			// contains the claim. Verify that the PV and PVC bind correctly, and
 			// that the pod can write to the nfs volume.
-			It("should create a non-pre-bound PV and PVC: test write access [Flaky]", func() {
+			It("should create a non-pre-bound PV and PVC: test write access [Volume][Serial][Flaky]", func() {
 				pv, pvc = createPVPVC(c, pvConfig, ns, false)
 				completeTest(f, c, ns, pv, pvc)
 			})
 
 			// Create a claim first, then a nfs PV that matches the claim, and a
 			// pod that contains the claim. Verify that the PV and PVC bind
 			// correctly, and that the pod can write to the nfs volume.
-			It("create a PVC and non-pre-bound PV: test write access [Flaky]", func() {
+			It("create a PVC and non-pre-bound PV: test write access [Volume][Serial][Flaky]", func() {
 				pv, pvc = createPVCPV(c, pvConfig, ns, false)
 				completeTest(f, c, ns, pv, pvc)
 			})
 
 			// Create a claim first, then a pre-bound nfs PV that matches the claim,
 			// and a pod that contains the claim. Verify that the PV and PVC bind
 			// correctly, and that the pod can write to the nfs volume.
-			It("create a PVC and a pre-bound PV: test write access [Flaky]", func() {
+			It("create a PVC and a pre-bound PV: test write access [Volume][Serial][Flaky]", func() {
 				pv, pvc = createPVCPV(c, pvConfig, ns, true)
 				completeTest(f, c, ns, pv, pvc)
 			})
 
 			// Create a nfs PV first, then a pre-bound PVC that matches the PV,
 			// and a pod that contains the claim. Verify that the PV and PVC bind
 			// correctly, and that the pod can write to the nfs volume.
-			It("create a PV and a pre-bound PVC: test write access [Flaky]", func() {
+			It("create a PV and a pre-bound PVC: test write access [Volume][Serial][Flaky]", func() {
 				pv, pvc = createPVPVC(c, pvConfig, ns, true)
 				completeTest(f, c, ns, pv, pvc)
 			})
@@ -615,7 +615,7 @@ var _ = framework.KubeDescribe("PersistentVolumes", func() {
 
 			// Create 2 PVs and 4 PVCs.
 			// Note: PVs are created before claims and no pre-binding
-			It("should create 2 PVs and 4 PVCs: test write access[Flaky]", func() {
+			It("should create 2 PVs and 4 PVCs: test write access [Volume][Serial][Flaky]", func() {
 				numPVs, numPVCs := 2, 4
 				pvols, claims = createPVsPVCs(numPVs, numPVCs, c, ns, pvConfig)
 				waitAndVerifyBinds(c, ns, pvols, claims, true)
@@ -624,7 +624,7 @@ var _ = framework.KubeDescribe("PersistentVolumes", func() {
 
 			// Create 3 PVs and 3 PVCs.
 			// Note: PVs are created before claims and no pre-binding
-			It("should create 3 PVs and 3 PVCs: test write access[Flaky]", func() {
+			It("should create 3 PVs and 3 PVCs: test write access [Volume][Serial][Flaky]", func() {
 				numPVs, numPVCs := 3, 3
 				pvols, claims = createPVsPVCs(numPVs, numPVCs, c, ns, pvConfig)
 				waitAndVerifyBinds(c, ns, pvols, claims, true)
@@ -633,7 +633,7 @@ var _ = framework.KubeDescribe("PersistentVolumes", func() {
 
 			// Create 4 PVs and 2 PVCs.
 			// Note: PVs are created before claims and no pre-binding.
-			It("should create 4 PVs and 2 PVCs: test write access[Flaky]", func() {
+			It("should create 4 PVs and 2 PVCs: test write access [Volume][Serial][Flaky]", func() {
 				numPVs, numPVCs := 4, 2
 				pvols, claims = createPVsPVCs(numPVs, numPVCs, c, ns, pvConfig)
 				waitAndVerifyBinds(c, ns, pvols, claims, true)
@@ -694,7 +694,7 @@ var _ = framework.KubeDescribe("PersistentVolumes", func() {
 
 		// Attach a persistent disk to a pod using a PVC.
 		// Delete the PVC and then the pod.  Expect the pod to succeed in unmounting and detaching PD on delete.
-		It("should test that deleting a PVC before the pod does not cause pod deletion to fail on PD detach", func() {
+		It("should test that deleting a PVC before the pod does not cause pod deletion to fail on PD detach [Volume][Serial][Flaky]", func() {
 			By("Creating the PV and PVC")
 			pv, pvc = createPVPVC(c, pvConfig, ns, false)
 			waitOnPVandPVC(c, ns, pv, pvc)
@@ -717,7 +717,7 @@ var _ = framework.KubeDescribe("PersistentVolumes", func() {
 
 		// Attach a persistent disk to a pod using a PVC.
 		// Delete the PV and then the pod.  Expect the pod to succeed in unmounting and detaching PD on delete.
-		It("should test that deleting the PV before the pod does not cause pod deletion to fail on PD detach", func() {
+		It("should test that deleting the PV before the pod does not cause pod deletion to fail on PD detach [Volume][Serial][Flaky]", func() {
 			By("Creating the PV and PVC")
 			pv, pvc = createPVPVC(c, pvConfig, ns, false)
 			waitOnPVandPVC(c, ns, pv, pvc)