Delete pod in volume tests

jsafrane · jsafrane · commit 2430c48c104f · 2020-02-11T12:54:38.000+01:00
All storage e2e tests should delete pods they use so we can identify issues
on volume cleanup easily.
diff --git a/test/e2e/framework/volume/fixtures.go b/test/e2e/framework/volume/fixtures.go
@@ -486,8 +486,12 @@ func TestVolumeClient(f *framework.Framework, config TestConfig, fsGroup *int64,
 	clientPod, err := runVolumeTesterPod(f.ClientSet, config, "client", false, fsGroup, tests)
 	if err != nil {
 		framework.Failf("Failed to create client pod: %v", err)
-
 	}
+	defer func() {
+		e2epod.DeletePodOrFail(f.ClientSet, clientPod.Namespace, clientPod.Name)
+		e2epod.WaitForPodToDisappear(f.ClientSet, clientPod.Namespace, clientPod.Name, labels.Everything(), framework.Poll, framework.PodDeleteTimeout)
+	}()
+
 	framework.ExpectNoError(e2epod.WaitForPodRunningInNamespace(f.ClientSet, clientPod))
 	testVolumeContent(f, clientPod, fsGroup, fsType, tests)
 }

Original file line number	Diff line number	Diff line change
`@@ -486,8 +486,12 @@ func TestVolumeClient(f framework.Framework, config TestConfig, fsGroup int64,`
`486`	`486`	`clientPod, err := runVolumeTesterPod(f.ClientSet, config, "client", false, fsGroup, tests)`
`487`	`487`	`if err != nil {`
`488`	`488`	`framework.Failf("Failed to create client pod: %v", err)`
`489`		`-`
`490`	`489`	`}`
	`490`	`+ defer func() {`
	`491`	`+ e2epod.DeletePodOrFail(f.ClientSet, clientPod.Namespace, clientPod.Name)`
	`492`	`+ e2epod.WaitForPodToDisappear(f.ClientSet, clientPod.Namespace, clientPod.Name, labels.Everything(), framework.Poll, framework.PodDeleteTimeout)`
	`493`	`+ }()`
	`494`	`+`
`491`	`495`	`framework.ExpectNoError(e2epod.WaitForPodRunningInNamespace(f.ClientSet, clientPod))`
`492`	`496`	`testVolumeContent(f, clientPod, fsGroup, fsType, tests)`
`493`	`497`	`}`