From 4491ba891719eff0b457c239eed8fbca0d78a393 Mon Sep 17 00:00:00 2001 From: Humble Chirammal Date: Tue, 11 Apr 2023 16:48:08 +0530 Subject: [PATCH 1/2] rename variable delete to avoid collision to builtin variable Signed-off-by: Humble Chirammal --- test/e2e/storage/csi_mock/csi_fsgroup_policy.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/e2e/storage/csi_mock/csi_fsgroup_policy.go b/test/e2e/storage/csi_mock/csi_fsgroup_policy.go index f57897856b1..abe61e00797 100644 --- a/test/e2e/storage/csi_mock/csi_fsgroup_policy.go +++ b/test/e2e/storage/csi_mock/csi_fsgroup_policy.go @@ -104,8 +104,8 @@ var _ = utils.SIGDescribe("CSI Mock volume fsgroup policies", func() { // Delete the created file. This step is mandatory, as the mock driver // won't clean up the contents automatically. defer func() { - delete := fmt.Sprintf("rm -fr %s", dirName) - _, _, err = e2evolume.PodExec(f, pod, delete) + deleteDir := fmt.Sprintf("rm -fr %s", dirName) + _, _, err = e2evolume.PodExec(f, pod, deleteDir) framework.ExpectNoError(err, "failed: deleting the directory: %s", err) }() From 9204e6b286ecacc2d38d60baf823fd3d6a7f7843 Mon Sep 17 00:00:00 2001 From: Humble Chirammal Date: Tue, 11 Apr 2023 16:48:56 +0530 Subject: [PATCH 2/2] remove unused constant `expansionFailed` in expansion test. Signed-off-by: Humble Chirammal --- test/e2e/storage/csi_mock/csi_volume_expansion.go | 1 - 1 file changed, 1 deletion(-) diff --git a/test/e2e/storage/csi_mock/csi_volume_expansion.go b/test/e2e/storage/csi_mock/csi_volume_expansion.go index cbc840f098d..9673ea75823 100644 --- a/test/e2e/storage/csi_mock/csi_volume_expansion.go +++ b/test/e2e/storage/csi_mock/csi_volume_expansion.go @@ -43,7 +43,6 @@ type expansionStatus int const ( expansionSuccess = iota - expansionFailed expansionFailedOnController expansionFailedOnNode expansionFailedMissingStagingPath