Change in ovirt-engine[master]: core: live snapshot failes in case of a deactivated disk (#8...

mlipchuk at redhat.com mlipchuk at redhat.com
Mon Aug 20 09:30:17 UTC 2012


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: live snapshot failes in case of a deactivated disk (#845020)
......................................................................


core: live snapshot failes in case of a deactivated disk (#845020)

https://bugzilla.redhat.com/845020

when creating live snapshot for a VM, a SnapshotVdsCommand was executed
for all disks, plugged and unplugged in order to switch the images that
the vm is running on to the newly created images.  in case of an
unplugged disks,
the command had failed as the VM doesn't currently use the unplugged
disk. this fix
filters out the unplugged disks from the list of disks sent to the
SnapshotVdsCommand.

Change-Id: I27c32525794b4ec3c9d8c78f033a3c9d39da72b9
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, but someone else must approve
  Maor Lipchuk: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I27c32525794b4ec3c9d8c78f033a3c9d39da72b9
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>



More information about the Engine-commits mailing list