diff --git a/test/e2e/storage/csi_mock_volume.go b/test/e2e/storage/csi_mock_volume.go index 595b8ee5215..39383ef3f2c 100644 --- a/test/e2e/storage/csi_mock_volume.go +++ b/test/e2e/storage/csi_mock_volume.go @@ -470,8 +470,9 @@ var _ = utils.SIGDescribe("CSI mock volume", func() { ginkgo.By("Expanding current pvc") newSize := resource.MustParse("6Gi") - pvc, err = testsuites.ExpandPVCSize(pvc, newSize, m.cs) + newPVC, err := testsuites.ExpandPVCSize(pvc, newSize, m.cs) framework.ExpectNoError(err, "While updating pvc for more size") + pvc = newPVC gomega.Expect(pvc).NotTo(gomega.BeNil()) pvcSize := pvc.Spec.Resources.Requests[v1.ResourceStorage] @@ -561,8 +562,9 @@ var _ = utils.SIGDescribe("CSI mock volume", func() { ginkgo.By("Expanding current pvc") newSize := resource.MustParse("6Gi") - pvc, err = testsuites.ExpandPVCSize(pvc, newSize, m.cs) + newPVC, err := testsuites.ExpandPVCSize(pvc, newSize, m.cs) framework.ExpectNoError(err, "While updating pvc for more size") + pvc = newPVC gomega.Expect(pvc).NotTo(gomega.BeNil()) pvcSize := pvc.Spec.Resources.Requests[v1.ResourceStorage] diff --git a/test/e2e/storage/flexvolume_mounted_volume_resize.go b/test/e2e/storage/flexvolume_mounted_volume_resize.go index 0a21de8d3bd..8696a4b501a 100644 --- a/test/e2e/storage/flexvolume_mounted_volume_resize.go +++ b/test/e2e/storage/flexvolume_mounted_volume_resize.go @@ -159,8 +159,9 @@ var _ = utils.SIGDescribe("Mounted flexvolume expand[Slow]", func() { ginkgo.By("Expanding current pvc") newSize := resource.MustParse("6Gi") - pvc, err = testsuites.ExpandPVCSize(pvc, newSize, c) + newPVC, err := testsuites.ExpandPVCSize(pvc, newSize, c) framework.ExpectNoError(err, "While updating pvc for more size") + pvc = newPVC gomega.Expect(pvc).NotTo(gomega.BeNil()) pvcSize := pvc.Spec.Resources.Requests[v1.ResourceStorage] diff --git a/test/e2e/storage/flexvolume_online_resize.go b/test/e2e/storage/flexvolume_online_resize.go index 535bad70e35..c055007069e 100644 --- a/test/e2e/storage/flexvolume_online_resize.go +++ b/test/e2e/storage/flexvolume_online_resize.go @@ -159,8 +159,9 @@ var _ = utils.SIGDescribe("Mounted flexvolume volume expand [Slow] [Feature:Expa ginkgo.By("Expanding current pvc") newSize := resource.MustParse("6Gi") - pvc, err = testsuites.ExpandPVCSize(pvc, newSize, c) + newPVC, err := testsuites.ExpandPVCSize(pvc, newSize, c) framework.ExpectNoError(err, "While updating pvc for more size") + pvc = newPVC gomega.Expect(pvc).NotTo(gomega.BeNil()) pvcSize := pvc.Spec.Resources.Requests[v1.ResourceStorage] diff --git a/test/e2e/storage/mounted_volume_resize.go b/test/e2e/storage/mounted_volume_resize.go index 24d76eee8ed..06066070bc6 100644 --- a/test/e2e/storage/mounted_volume_resize.go +++ b/test/e2e/storage/mounted_volume_resize.go @@ -131,8 +131,9 @@ var _ = utils.SIGDescribe("Mounted volume expand", func() { ginkgo.By("Expanding current pvc") newSize := resource.MustParse("6Gi") - pvc, err = testsuites.ExpandPVCSize(pvc, newSize, c) + newPVC, err := testsuites.ExpandPVCSize(pvc, newSize, c) framework.ExpectNoError(err, "While updating pvc for more size") + pvc = newPVC gomega.Expect(pvc).NotTo(gomega.BeNil()) pvcSize := pvc.Spec.Resources.Requests[v1.ResourceStorage] diff --git a/test/e2e/storage/testsuites/volume_expand.go b/test/e2e/storage/testsuites/volume_expand.go index 8d73a4539b0..7accadd3155 100644 --- a/test/e2e/storage/testsuites/volume_expand.go +++ b/test/e2e/storage/testsuites/volume_expand.go @@ -146,7 +146,7 @@ func (v *volumeExpandTestSuite) defineTests(driver TestDriver, pattern testpatte newSize := currentPvcSize.DeepCopy() newSize.Add(resource.MustParse("1Gi")) e2elog.Logf("currentPvcSize %v, newSize %v", currentPvcSize, newSize) - l.resource.pvc, err = ExpandPVCSize(l.resource.pvc, newSize, f.ClientSet) + _, err = ExpandPVCSize(l.resource.pvc, newSize, f.ClientSet) framework.ExpectError(err, "While updating non-expandable PVC") }) } else { @@ -173,8 +173,9 @@ func (v *volumeExpandTestSuite) defineTests(driver TestDriver, pattern testpatte newSize := currentPvcSize.DeepCopy() newSize.Add(resource.MustParse("1Gi")) e2elog.Logf("currentPvcSize %v, newSize %v", currentPvcSize, newSize) - l.resource.pvc, err = ExpandPVCSize(l.resource.pvc, newSize, f.ClientSet) + newPVC, err := ExpandPVCSize(l.resource.pvc, newSize, f.ClientSet) framework.ExpectNoError(err, "While updating pvc for more size") + l.resource.pvc = newPVC gomega.Expect(l.resource.pvc).NotTo(gomega.BeNil()) pvcSize := l.resource.pvc.Spec.Resources.Requests[v1.ResourceStorage] @@ -232,8 +233,9 @@ func (v *volumeExpandTestSuite) defineTests(driver TestDriver, pattern testpatte newSize := currentPvcSize.DeepCopy() newSize.Add(resource.MustParse("1Gi")) e2elog.Logf("currentPvcSize %v, newSize %v", currentPvcSize, newSize) - l.resource.pvc, err = ExpandPVCSize(l.resource.pvc, newSize, f.ClientSet) + newPVC, err := ExpandPVCSize(l.resource.pvc, newSize, f.ClientSet) framework.ExpectNoError(err, "While updating pvc for more size") + l.resource.pvc = newPVC gomega.Expect(l.resource.pvc).NotTo(gomega.BeNil()) pvcSize := l.resource.pvc.Spec.Resources.Requests[v1.ResourceStorage]