From 9b8bcbd874cbb9da963a0eb164eadd8aad62b8ad Mon Sep 17 00:00:00 2001 From: Humble Chirammal Date: Mon, 11 Apr 2022 13:50:27 +0530 Subject: [PATCH] e2e: correct the cephfs fuse verification podnames The podnames were wrongly mention in the e2e tests and this correct the same. Signed-off-by: Humble Chirammal --- e2e/cephfs.go | 8 ++++---- e2e/kms.go | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/e2e/cephfs.go b/e2e/cephfs.go index b59827b9d..4b42b75a1 100644 --- a/e2e/cephfs.go +++ b/e2e/cephfs.go @@ -456,7 +456,7 @@ var _ = Describe("cephfs", func() { // clean up after ourselves err = deletePVCAndValidatePV(f.ClientSet, pvc, deployTimeout) if err != nil { - e2elog.Failf("failed to delete PVC: %v", err) + e2elog.Failf("failed to delete PVC: %v", err) } validateSubvolumeCount(f, 0, fileSystemName, subvolumegroup) err = deleteResource(cephFSExamplePath + "storageclass.yaml") @@ -575,7 +575,7 @@ var _ = Describe("cephfs", func() { if err != nil { e2elog.Failf("killall command failed: err %v, stderr %s", err, stdErr) } - // Verify Pod podName2 that stat()-ing the mountpoint results in ENOTCONN. + // Verify Pod pod2Name that stat()-ing the mountpoint results in ENOTCONN. stdErr, err = doStat(pod2Name) if err == nil || !strings.Contains(stdErr, "not connected") { e2elog.Failf( @@ -583,7 +583,7 @@ var _ = Describe("cephfs", func() { err, stdErr, ) } - // Delete podName2 Pod. This serves two purposes: it verifies that deleting pods with + // Delete pod2Name Pod. This serves two purposes: it verifies that deleting pods with // corrupted ceph-fuse mountpoints works, and it lets the replicaset controller recreate // the pod with hopefully mounts working again. err = deletePod(pod2Name, depl.Namespace, c, deployTimeout) @@ -616,7 +616,7 @@ var _ = Describe("cephfs", func() { } e2elog.Failf("no new replica found ; found replicas %v", podNames) } - // Verify Pod podName3 has its ceph-fuse mount working again. + // Verify Pod pod2Name has its ceph-fuse mount working again. err = ensureStatSucceeds(pod2Name) if err != nil { e2elog.Failf(err.Error()) diff --git a/e2e/kms.go b/e2e/kms.go index 0d3ae9655..7cf1a74e0 100644 --- a/e2e/kms.go +++ b/e2e/kms.go @@ -31,7 +31,7 @@ const ( // kmsConfig is an interface that should be used when passing a configuration // for a KMS to validation functions. This allows the validation functions to -// work independently from the actual KMS. +// work independently of the actual KMS. type kmsConfig interface { canGetPassphrase() bool getPassphrase(f *framework.Framework, key string) (string, string)