Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Refreshing ESX 6.5 support on CI #879

Conversation

shuklanirdesh82
Copy link
Contributor

CI is refreshed to unblock #859.

TestConcurrency is commented out to take further triage.

=== RUN   TestConcurrency
Running concurrent tests on tcp://192.168.31.187:2375 and tcp://192.168.31.83:2375 (may take a while)...
Running create/delete concurrent test...
--- FAIL: TestConcurrency (29.75s)
	sanity_test.go:199: Successfully connected to tcp://192.168.31.187:2375
	sanity_test.go:199: Successfully connected to tcp://192.168.31.83:2375
	sanity_test.go:298: Create/delete concurrent test failed, err: Error response from daemon: create volTestP10: Post http://%2Frun%2Fdocker%2Fplugins%2Fvmdk.sock/VolumeDriver.Create: http: ContentLength=44 with Body length 0
FAIL

vmdk_ops log

01/27/17 12:47:23 558076 [Ubuntu.14.04-[vsanDatastore] dockvols/4af40ea9-ae91-4556-b6b6-641b72c85ec5/volTestP10.vmdk] [WARNING] vmci_reply returned error Broken pipe (errno=32)
01/27/17 12:47:25 558076 [photon.vsan-[vsanDatastore] dockvols/4af40ea9-ae91-4556-b6b6-641b72c85ec5/volTestP00.vmdk] [INFO   ] executeRequest 'remove' completed with ret=None
01/27/17 12:47:25 558076 [Thread-39] [INFO   ] cmd get with opts {} on tenant_uuid 4af40ea9-ae91-4556-b6b6-641b72c85ec5 datastore vsanDatastore is allowed to execute
01/27/17 12:47:25 558076 [photon.vsan-[vsanDatastore] dockvols/4af40ea9-ae91-4556-b6b6-641b72c85ec5/volTestP01.vmdk] [INFO   ] executeRequest 'get' completed with ret={'Error': 'Volume volTestP01 not found (file: /vmfs/volumes/vsanDatastore/dockvols/4af40ea9-ae91-4556-b6b6-641b72c85ec5/volTestP01.vmdk)'}

//CC @shaominchen

@shaominchen
Copy link
Contributor

@shuklanirdesh82 Do you plan to check in this change first, and investigate the TestConcurrency failure separately?

@shuklanirdesh82
Copy link
Contributor Author

Yeah @shaominchen that's the plan, the failure is observed only against 1.13 docker and will open the separate issue later on. After getting approval you can get the change and resume your testing.

The CI changes are ready for you to test out your change.

FYI: The failure observed with my proposed change is different and expected.

@shaominchen
Copy link
Contributor

@shuklanirdesh82 As I talked to you offline, I've already finished all my tests on both 6.5 and 6.0U2 setups. But until new CI is ready, I can't merge my change because it will fail in the current CI.

So, I don't need to test out my change - everything is done. Just need the new CI ready so that I can merge my PR.

@shuklanirdesh82 shuklanirdesh82 force-pushed the CIVsanIssue.shuklanirdesh82 branch 3 times, most recently from dd5c003 to e55a72a Compare January 29, 2017 00:56
Copy link
Contributor

@kerneltime kerneltime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which issue is tracking the uncommenting of the tests here?

@kerneltime
Copy link
Contributor

Feel free to merge but link the issue where track uncommenting the tests commented here.

@shuklanirdesh82
Copy link
Contributor Author

Thanks @kerneltime for the approval, reference has been raised to uncomment.

@shuklanirdesh82 shuklanirdesh82 deleted the CIVsanIssue.shuklanirdesh82 branch January 30, 2017 19:20
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants