Fail test when Cinder volume deletion fails

jsafrane · jsafrane · commit 76254d88a590 · 2020-09-23T14:41:29.000+02:00
diff --git a/test/e2e/storage/drivers/in_tree.go b/test/e2e/storage/drivers/in_tree.go
@@ -1216,7 +1216,7 @@ func (v *cinderVolume) DeleteVolume() {
 	} else {
 		framework.Logf("Volume %s / %s:\n%s", id, name, string(showOutput))
 	}
-	framework.Logf("Giving up deleting volume %s / %s: %v\n%s", id, name, err, string(output[:]))
+	framework.Failf("Failed to delete pre-provisioned volume %s / %s: %v\n%s", id, name, err, string(output[:]))
 }
 
 // GCE

Original file line number	Diff line number	Diff line change
`@@ -1216,7 +1216,7 @@ func (v *cinderVolume) DeleteVolume() {`
`1216`	`1216`	`} else {`
`1217`	`1217`	`framework.Logf("Volume %s / %s:\n%s", id, name, string(showOutput))`
`1218`	`1218`	`}`
`1219`		`- framework.Logf("Giving up deleting volume %s / %s: %v\n%s", id, name, err, string(output[:]))`
	`1219`	`+ framework.Failf("Failed to delete pre-provisioned volume %s / %s: %v\n%s", id, name, err, string(output[:]))`
`1220`	`1220`	`}`
`1221`	`1221`
`1222`	`1222`	`// GCE`