search

vmware-archive / vsphere-storage-for-docker

vSphere Storage for Docker
https://vmware.github.io/vsphere-storage-for-docker
Apache License 2.0
251 stars 95 forks source link

Move IsVDVSRunningAfterVMRestart to the util/docker #1572

Closed govint closed 6 years ago

govint commented 7 years ago

IsVDVSRunningAfterVMRestart() is added to utils/esx via #1563, but the managed plugin should be moved to utils/docker as its a docker managed plugin and it also needs to check that docker itself is up after a VM is powered on besides checking for just the plugin.

The check itself is weak, it checks if a process exists for the plugin, while it takes a good 20 seconds after the plugin starts for it to be ready to handle mounts/unmounts reliably. There has to be a way to (via the docker volume commands) to be able to figure if the plugin is really ready.

govint commented 7 years ago

Moving to spiderman as this is not a bug and not breaking anything for the e2e tests.

govint commented 6 years ago

Test issue and not a bug. Closing.