Dan Kenigsberg has submitted this change and it was merged.

Change subject: virt: Resolve snapshot type after prepareVolumePath
......................................................................


virt: Resolve snapshot type after prepareVolumePath

The Drive.blockDev property depends on the presence of the volume path
locally.  Therefore, it is not safe to call until after the new snapshot
volume path has been prepared.  Use a lambda function to delay the call
until the information is needed.

Change-Id: Ia4726702eb329bf65f1f9c658067f7a01c2179e4
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109920
Signed-off-by: Adam Litke <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/32066
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Federico Simoncelli <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/32327
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M vdsm/virt/vm.py
1 file changed, 7 insertions(+), 3 deletions(-)

Approvals:
  Adam Litke: Verified
  Federico Simoncelli: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/32327
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia4726702eb329bf65f1f9c658067f7a01c2179e4
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Adam Litke <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to