From d10179920347d3aba1dfb06aebc917207663ced1 Mon Sep 17 00:00:00 2001 From: Venil Noronha Date: Mon, 9 Apr 2018 11:23:58 -0700 Subject: [PATCH] Addresses review comments --- test/e2e/storage/vsphere/vsphere_volume_vpxd_restart.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/e2e/storage/vsphere/vsphere_volume_vpxd_restart.go b/test/e2e/storage/vsphere/vsphere_volume_vpxd_restart.go index 973a5f4ffa0..e30704ab3cb 100644 --- a/test/e2e/storage/vsphere/vsphere_volume_vpxd_restart.go +++ b/test/e2e/storage/vsphere/vsphere_volume_vpxd_restart.go @@ -68,7 +68,9 @@ var _ = utils.SIGDescribe("Verify Volume Attach Through vpxd Restart [Feature:vs ) BeforeEach(func() { + // Requires SSH access to vCenter. framework.SkipUnlessProviderIs("vsphere") + Bootstrap(f) client = f.ClientSet namespace = f.Namespace.Name @@ -76,9 +78,7 @@ var _ = utils.SIGDescribe("Verify Volume Attach Through vpxd Restart [Feature:vs nodes := framework.GetReadySchedulableNodesOrDie(client) numNodes := len(nodes.Items) - if numNodes < 1 { - framework.Skipf("Requires at least %d nodes (not %d)", 1, len(nodes.Items)) - } + Expect(numNodes).NotTo(BeZero(), "No nodes are available for testing volume access through vpxd restart") vcNodesMap = make(map[string][]node) for i := 0; i < numNodes; i++ {