Change in ovirt-engine[master]: core: Snapshot should not be update after rollback.
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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>