e2e: remove 1.15 based test enablement in rbd

considering we are far out of this release and only care about kubernetes releases from v1.19, there is no need to have this version check in place for the tests. Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
ceph · Nov 3, 2021 · be7f407 · be7f407
1 parent 32b4c1e
commit be7f407
Showing 1 changed file with 65 additions and 80 deletions.
diff --git a/e2e/rbd.go b/e2e/rbd.go
@@ -1105,7 +1105,6 @@ var _ = Describe("RBD", func() {
 					noKMS, noKMS,
 					defaultSCName,
 					f)
-
 			})
 
 			By("create a PVC-PVC clone and bind it to an app", func() {
@@ -1118,7 +1117,6 @@ var _ = Describe("RBD", func() {
 					noKMS,
 					noPVCValidation,
 					f)
-
 			})
 
 			By("create a thick-provisioned PVC-PVC clone and bind it to an app", func() {
@@ -1419,34 +1417,31 @@ var _ = Describe("RBD", func() {
 			})
 
 			By("Resize Filesystem PVC and check application directory size", func() {
-				// Resize 0.3.0 is only supported from v1.15+
-				if k8sVersionGreaterEquals(f.ClientSet, 1, 15) {
-					err := resizePVCAndValidateSize(pvcPath, appPath, f)
-					if err != nil {
-						e2elog.Failf("failed to resize filesystem PVC %v", err)
-					}
+				err := resizePVCAndValidateSize(pvcPath, appPath, f)
+				if err != nil {
+					e2elog.Failf("failed to resize filesystem PVC %v", err)
+				}
 
-					err = deleteResource(rbdExamplePath + "storageclass.yaml")
-					if err != nil {
-						e2elog.Failf("failed to delete storageclass with error %v", err)
-					}
-					err = createRBDStorageClass(
-						f.ClientSet,
-						f,
-						defaultSCName,
-						nil,
-						map[string]string{"csi.storage.k8s.io/fstype": "xfs"},
-						deletePolicy)
-					if err != nil {
-						e2elog.Failf("failed to create storageclass with error %v", err)
-					}
-					err = resizePVCAndValidateSize(pvcPath, appPath, f)
-					if err != nil {
-						e2elog.Failf("failed to resize filesystem PVC with error %v", err)
-					}
-					// validate created backend rbd images
-					validateRBDImageCount(f, 0, defaultRBDPool)
+				err = deleteResource(rbdExamplePath + "storageclass.yaml")
+				if err != nil {
+					e2elog.Failf("failed to delete storageclass with error %v", err)
 				}
+				err = createRBDStorageClass(
+					f.ClientSet,
+					f,
+					defaultSCName,
+					nil,
+					map[string]string{"csi.storage.k8s.io/fstype": "xfs"},
+					deletePolicy)
+				if err != nil {
+					e2elog.Failf("failed to create storageclass with error %v", err)
+				}
+				err = resizePVCAndValidateSize(pvcPath, appPath, f)
+				if err != nil {
+					e2elog.Failf("failed to resize filesystem PVC with error %v", err)
+				}
+				// validate created backend rbd images
+				validateRBDImageCount(f, 0, defaultRBDPool)
 			})
 
 			By("Resize Block PVC and check Device size", func() {
@@ -2020,7 +2015,6 @@ var _ = Describe("RBD", func() {
 				}
 				// validate created backend rbd images
 				validateRBDImageCount(f, 0, defaultRBDPool)
-
 			})
 
 			By("validate PVC mounting if snapshot and parent PVC are deleted", func() {
@@ -2453,12 +2447,9 @@ var _ = Describe("RBD", func() {
 				}
 
 				// Resize Filesystem PVC and check application directory size
-				// Resize 0.3.0 is only supported from v1.15+
-				if k8sVersionGreaterEquals(f.ClientSet, 1, 15) {
-					err = resizePVCAndValidateSize(pvcPath, appPath, f)
-					if err != nil {
-						e2elog.Failf("failed to resize filesystem PVC %v", err)
-					}
+				err = resizePVCAndValidateSize(pvcPath, appPath, f)
+				if err != nil {
+					e2elog.Failf("failed to resize filesystem PVC %v", err)
 				}
 
 				// Create a PVC clone and bind it to an app within the namespace
@@ -2469,58 +2460,52 @@ var _ = Describe("RBD", func() {
 				defer func() {
 					err = deleteRBDSnapshotClass()
 					if err != nil {
-						e2elog.Failf("failed to create storageclass with error %v", err)
+						e2elog.Failf("failed to delete VolumeSnapshotClass: %v", err)
 					}
-					defer func() {
-						err = deleteRBDSnapshotClass()
-						if err != nil {
-							e2elog.Failf("failed to delete VolumeSnapshotClass: %v", err)
-						}
-					}()
+				}()
 
-					pvc, pvcErr := loadPVC(pvcPath)
-					if pvcErr != nil {
-						e2elog.Failf("failed to load PVC with error %v", pvcErr)
-					}
+				pvc, pvcErr := loadPVC(pvcPath)
+				if pvcErr != nil {
+					e2elog.Failf("failed to load PVC with error %v", pvcErr)
+				}
 
-					pvc.Namespace = f.UniqueName
-					err = createPVCAndvalidatePV(f.ClientSet, pvc, deployTimeout)
-					if err != nil {
-						e2elog.Failf("failed to create PVC with error %v", err)
-					}
-					// validate created backend rbd images
-					validateRBDImageCount(f, 1, defaultRBDPool)
+				pvc.Namespace = f.UniqueName
+				err = createPVCAndvalidatePV(f.ClientSet, pvc, deployTimeout)
+				if err != nil {
+					e2elog.Failf("failed to create PVC with error %v", err)
+				}
+				// validate created backend rbd images
+				validateRBDImageCount(f, 1, defaultRBDPool)
 
-					snap := getSnapshot(snapshotPath)
-					snap.Namespace = f.UniqueName
-					snap.Spec.Source.PersistentVolumeClaimName = &pvc.Name
-					err = createSnapshot(&snap, deployTimeout)
-					if err != nil {
-						e2elog.Failf("failed to create snapshot with error %v", err)
-					}
-					validateRBDImageCount(f, 2, defaultRBDPool)
+				snap := getSnapshot(snapshotPath)
+				snap.Namespace = f.UniqueName
+				snap.Spec.Source.PersistentVolumeClaimName = &pvc.Name
+				err = createSnapshot(&snap, deployTimeout)
+				if err != nil {
+					e2elog.Failf("failed to create snapshot with error %v", err)
+				}
+				validateRBDImageCount(f, 2, defaultRBDPool)
 
-					err = validatePVCAndAppBinding(pvcClonePath, appClonePath, f)
-					if err != nil {
-						e2elog.Failf("failed to validate pvc and application binding with error %v", err)
-					}
-					err = deleteSnapshot(&snap, deployTimeout)
-					if err != nil {
-						e2elog.Failf("failed to delete snapshot with error %v", err)
-					}
-					// as snapshot is deleted the image count should be one
-					validateRBDImageCount(f, 1, defaultRBDPool)
+				err = validatePVCAndAppBinding(pvcClonePath, appClonePath, f)
+				if err != nil {
+					e2elog.Failf("failed to validate pvc and application binding with error %v", err)
+				}
+				err = deleteSnapshot(&snap, deployTimeout)
+				if err != nil {
+					e2elog.Failf("failed to delete snapshot with error %v", err)
+				}
+				// as snapshot is deleted the image count should be one
+				validateRBDImageCount(f, 1, defaultRBDPool)
 
-					err = deletePVCAndValidatePV(f.ClientSet, pvc, deployTimeout)
-					if err != nil {
-						e2elog.Failf("failed to delete PVC with error %v", err)
-					}
-					validateRBDImageCount(f, 0, defaultRBDPool)
+				err = deletePVCAndValidatePV(f.ClientSet, pvc, deployTimeout)
+				if err != nil {
+					e2elog.Failf("failed to delete PVC with error %v", err)
+				}
+				validateRBDImageCount(f, 0, defaultRBDPool)
 
-					err = waitToRemoveImagesFromTrash(f, defaultRBDPool, deployTimeout)
-					if err != nil {
-						e2elog.Failf("failed to validate rbd images in pool %s trash with error %v", rbdOptions(defaultRBDPool), err)
-					}
+				err = waitToRemoveImagesFromTrash(f, defaultRBDPool, deployTimeout)
+				if err != nil {
+					e2elog.Failf("failed to validate rbd images in pool %s trash with error %v", rbdOptions(defaultRBDPool), err)
 				}
 
 				// delete RBD provisioner secret