Change in ovirt-engine[master]: core: errors during preview of diskless snapshots

[mlipchuk at redhat.com]

Maor Lipchuk has submitted this change and it was merged.

Change subject: core: errors during preview of diskless snapshots
......................................................................


core: errors during preview of diskless snapshots

1. When creating a diskless snapshot, then adding a disk to the vm and
later on try to preview that snapshots an NPE occurs within ovf writer.
The exception occurs because of outdated disk information contained within the
vm - the problem wasn't fixed locally but globally to prevent it from
occuring in other flows as well. VmHandler.updateDisksFromDb() should
clear the current disks of the vm otherwise we might get corrupted disk
information for the vm (by adding the disks information from the DB
while not clearing the present information).

2. When trying to run diskless snapshots exception can occur for various
reason in VmHandler.updateVmInSpm(). When such exception occurs the
operation should be rolled back (in the bug scenario - it wasn't as
there were db updates that were executed with no transaction which led
to partially rollback).

Change-Id: I6e8edc74bc34676f526dfd24d2f89eb60d8acf2e
Bug-Url: https://bugzilla.redhat.com/873595
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TryBackToAllSnapshotsOfVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
2 files changed, 16 insertions(+), 12 deletions(-)

Approvals:
  Maor Lipchuk: Looks good to me, approved
  Liron Aravot: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6e8edc74bc34676f526dfd24d2f89eb60d8acf2e
Gerrit-PatchSet: 5
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>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>