Commit d1017992 authored by Venil Noronha's avatar Venil Noronha

Addresses review comments

parent e90246d6
...@@ -68,7 +68,9 @@ var _ = utils.SIGDescribe("Verify Volume Attach Through vpxd Restart [Feature:vs ...@@ -68,7 +68,9 @@ var _ = utils.SIGDescribe("Verify Volume Attach Through vpxd Restart [Feature:vs
) )
BeforeEach(func() { BeforeEach(func() {
// Requires SSH access to vCenter.
framework.SkipUnlessProviderIs("vsphere") framework.SkipUnlessProviderIs("vsphere")
Bootstrap(f) Bootstrap(f)
client = f.ClientSet client = f.ClientSet
namespace = f.Namespace.Name namespace = f.Namespace.Name
...@@ -76,9 +78,7 @@ var _ = utils.SIGDescribe("Verify Volume Attach Through vpxd Restart [Feature:vs ...@@ -76,9 +78,7 @@ var _ = utils.SIGDescribe("Verify Volume Attach Through vpxd Restart [Feature:vs
nodes := framework.GetReadySchedulableNodesOrDie(client) nodes := framework.GetReadySchedulableNodesOrDie(client)
numNodes := len(nodes.Items) numNodes := len(nodes.Items)
if numNodes < 1 { Expect(numNodes).NotTo(BeZero(), "No nodes are available for testing volume access through vpxd restart")
framework.Skipf("Requires at least %d nodes (not %d)", 1, len(nodes.Items))
}
vcNodesMap = make(map[string][]node) vcNodesMap = make(map[string][]node)
for i := 0; i < numNodes; i++ { for i := 0; i < numNodes; i++ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment