Change in ovirt-engine[ovirt-engine-4.1]: core: avoid unlocking in-preview snapshot on failure

Code Review gerrit at ovirt.org
Wed Mar 15 13:20:44 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: avoid unlocking in-preview snapshot on failure
......................................................................


core: avoid unlocking in-preview snapshot on failure

Failure during RestoreAllSnapshotsCommand execution unlocks the
'In Preview' snapshot while keeping the 'Active VM before the
preview' snapshot. This allows previewing the snapshot again,
thus, making another 'preview' snapshot and a lot of mess.

The suggested solution is to simply avoid unlocking the snapshot
on endWithFailure.

Since the current logic exists from 2013 and seems invalid now,
I'm open for any other suggestions :)

Change-Id: I94c0b30b7621b8831f13815b3fdd8cdbce0117b7
Bug-Url: https://bugzilla.redhat.com/1427104
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RestoreAllSnapshotsCommand.java
1 file changed, 6 insertions(+), 1 deletion(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Daniel Erez: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/74119
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I94c0b30b7621b8831f13815b3fdd8cdbce0117b7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list