uncomment check to verify vm name is same on esx/docker host as reported by govc

[ashahi1]

Uncomment the code to verify vm name is same on esx and docker host as reported by govc
#1284 (comment)

Ran all the e2e-tests and all tests passed other than swarm test. Swarm tests failed because I wasn't having swarm setup.

=> Running target test-e2e-runalways
OK: 6 passed
--- PASS: Test (111.40s)
PASS
ok  	github.com/vmware/docker-volume-vsphere/tests/e2e	111.401s

=> Running target test-e2e-runonce
OOPS: 21 passed, 3 skipped, 2 FAILED, 1 MISSED
--- FAIL: Test (401.19s)
FAIL
exit status 1
FAIL	github.com/vmware/docker-volume-vsphere/tests/e2e	401.198s

[lipingxue]

What is the purpose of this PR?

[ashahi1]

@lipingxue #1284 (comment)

So I am uncommenting the lines which were commented out in PR #1276

[shaominchen]

@ashahi1 We shouldn't disable swarm test. The real issue here is with RetrieveVMNameFromIP itself. Please investigate further before re-enabling this function. I don't think we have truly figured out the root cause of the previous issue that we have identified about RetrieveVMNameFromIP.

[ashahi1]

@shaominchen We already know swarm test has test issue - Issue #1490
Disabled swarm test because need to find out if other tests have similar test issues.

[ashahi1]

It looks like only swarm test fails after we uncomment the lines in VerifyAttachedStatus method. Will wait for CI run to finish.
So, for this PR to merge, issue #1490 needs to be fixed first.
https://github.com/vmware/docker-volume-vsphere/blob/master/tests/utils/verification/volumeproperties.go#L104

/cc @shuklanirdesh82

[shuklanirdesh82]

Let's enable the check.

By taking reference of #1487 (comment) .. It is good to enable the check as only one test is blocking this enablement.

[ashahi1]

As per offline discussion with Sam, added unstable tag to swarm test so that swarm test does not run on CI till issue #1490 is fixed.

/cc @shaominchen