Daniel Erez has posted comments on this change.
Change subject: core: VmReplicateDiskFinish on SyncImageData failure
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/VmReplicateDiskFinishTaskHandler.java
Line 159: }
Line 160:
Line 161: @Override
Line 162: protected void revertTask() {
Line 163: // Preventing rollback on VmReplicateDiskFinish success
The rollback prevention is done by examining the db (in every case of failure,
e.g. engine crash). I.e. if VmReplicateDiskFinish succeeds we won't invoke
rollback since the db should be updated on success.
Line 164: // (checks whether the disk moved successfully to the target
storage domain)
Line 165: Guid targetStorageDomainId =
getEnclosingCommand().getParameters().getTargetStorageDomainId();
Line 166: Guid destinationImageId =
getEnclosingCommand().getParameters().getDestinationImageId();
Line 167: DiskImage diskImage =
getDiskImageDao().get(destinationImageId);
--
To view, visit http://gerrit.ovirt.org/14722
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6c25b02cb0c2bd78b012dd3e3654caa43f5141d6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
