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

GetVMAttachedToVolUsingAdminCli() should allow for same volume name on different datastores #1510

Closed
govint opened this issue Jun 29, 2017 · 1 comment

Comments

@govint
Copy link
Contributor

govint commented Jun 29, 2017

When testing with restarting docker with two volumes of the same name on different datastores, GetVMAttachedToVolUsingAdminCli() returned an error that the expected output had more than one match.

Its ok to have multiple volumes of the same name and on different datastores. GetVMAttachedToVolUsingAdminCli() should check the datastore as well, otherwise some tests don't necessarily work ok, with a false alarm.

2017/06/28 20:32:46 Attaching volume [restart_test_volume_666978] on VM[192.168.31.85]
2017/06/28 20:32:49 Confirming attached status for volume [restart_test_volume_666978]
2017/06/28 20:32:52 Admin cli output is expected to consist of two elements only - volume name and attached-to-vm status. Actual output
restart_test_volume_666978 detached <------- this volume is on datastore1
restart_test_volume_666978 Ubuntu.16.10 <------ this volume is on vsandatastore
exit status 1

@govint
Copy link
Contributor Author

govint commented Jul 10, 2017

Will be fixed via #1566

@govint govint closed this as completed Jul 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant