Merge pull request #81107 from wongma7/expand-cleanup

Don't overwrite PVC with nil/empty object returned by failed Update
This commit is contained in:
Kubernetes Prow Robot 2019-08-23 13:31:31 -07:00 committed by GitHub
commit 220498b83a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 15 additions and 8 deletions

View File

@ -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]

View File

@ -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]

View File

@ -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]

View File

@ -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]

View File

@ -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]