Change in ovirt-engine[master]: core: Snapshot should not be update after rollback.

[mlipchuk at redhat.com]

Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Snapshot should not be update after rollback.
......................................................................


core: Snapshot should not be update after rollback.

Once createAllSnapshotCommand execution encountered a rollback,
The new snapshot is deleted as part of the compensation scenario.

That will cause the end method to get null value for the snapshotId and
will cause an sql exception when it will be updated.

The update phase should be only when the tasks failed but
also when command finished successfully and the snapshot was already
created in the DB.

Change-Id: Ia96cd0958dcdb34d590a160b422b3b2562149344
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
Bug-Url: https://bugzilla.redhat.com/917719
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Maor Lipchuk: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia96cd0958dcdb34d590a160b422b3b2562149344
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>