Dan Kenigsberg has posted comments on this change.
Change subject: vm: discover volume path from xml definition ......................................................................
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/24202/2//COMMIT_MSG Commit Message:
Line 22: domain Line 23: Line 24: In this patch: Line 25: Line 26: - revert to the previous path for virtual machine images An explanation on the need to revert was not added.
But since this reversal is going to be just as harmful for current ovirt-3.3.1 users, as keeping the current path is for ovirt-3.3.0 upgrading, I am not sure that we should take that path.
How about splitting this to a different patch, and considering to use it only on systems with a big fleet of running vm that use the /rhev/dc/spUUID path. Line 27: - inspect libvirt xml during live migration and vdsm restart to Line 28: identify if it is necessary to update the path cached in the Line 29: drive object (provided by prepareImage) Line 30: