Skip to content

Commit 5ff5dbb

Browse files
authored
Merge pull request kubernetes#92497 from oomichi/vsphere
nit: Fix invokeStaleDummyVMTestWithStoragePolicy()
2 parents 0eaa854 + d964569 commit 5ff5dbb

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

test/e2e/storage/vsphere/vsphere_volume_vsan_policy.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -335,8 +335,9 @@ func invokeStaleDummyVMTestWithStoragePolicy(client clientset.Interface, masterN
335335
fnvHash := fnv.New32a()
336336
fnvHash.Write([]byte(vmName))
337337
dummyVMFullName := dummyVMPrefixName + "-" + fmt.Sprint(fnvHash.Sum32())
338-
errorMsg := "Dummy VM - " + vmName + "is still present. Failing the test.."
338+
errorMsg := "Dummy VM - " + vmName + " is still present. Failing the test.."
339339
nodeInfo := TestContext.NodeMapper.GetNodeInfo(masterNode)
340-
isVMPresentFlag, _ := nodeInfo.VSphere.IsVMPresent(dummyVMFullName, nodeInfo.DataCenterRef)
341-
framework.ExpectNotEqual(isVMPresentFlag, true, errorMsg)
340+
isVMPresentFlag, err := nodeInfo.VSphere.IsVMPresent(dummyVMFullName, nodeInfo.DataCenterRef)
341+
framework.ExpectNoError(err)
342+
framework.ExpectEqual(isVMPresentFlag, false, errorMsg)
342343
}

0 commit comments

Comments
 (0)