Liron Ar has posted comments on this change.
Change subject: core: remove vm - remove roll forward
......................................................................
Patch Set 1: (5 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveAllVmImagesCommand.java
Line 46: mImagesToBeRemoved.add(image.getImageId());
Line 47: }
Line 48: }
Line 49:
Line 50: Collection<DiskImage> failedToBeRemoved = new LinkedList<>();
Done
Line 51: for (final DiskImage image : images) {
Line 52: if (mImagesToBeRemoved.contains(image.getImageId())) {
Line 53: VdcReturnValueBase vdcReturnValue =
Line 54:
Backend.getInstance().runInternalAction(VdcActionType.RemoveImage,
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java
Line 94: return null;
Line 95: }
Line 96: });
Line 97:
Line 98: if (getParameters().isRemoveDisks() && hasImages) {
yeah, that was here before..the if clause was just copied from line 79..but ok,
i removed it.
Line 99: Collection<DiskImage> unremovedDisks =
(Collection<DiskImage>)removeVmImages(diskImages).getActionReturnValue();
Line 100: if (!unremovedDisks.isEmpty()) {
Line 101: processUnremovedDisks(unremovedDisks);
Line 102: setSucceeded(false);
Line 98: if (getParameters().isRemoveDisks() && hasImages) {
Line 99: Collection<DiskImage> unremovedDisks =
(Collection<DiskImage>)removeVmImages(diskImages).getActionReturnValue();
Line 100: if (!unremovedDisks.isEmpty()) {
Line 101: processUnremovedDisks(unremovedDisks);
Line 102: setSucceeded(false);
Done
Line 103: }
Line 104: }
Line 105:
Line 106: return true;
Line 242:
Line 243: @Override
Line 244: public AuditLogType getAuditLogTypeValue() {
Line 245: return getSucceeded() ? (disksLeftInVm.isEmpty() ?
AuditLogType.USER_REMOVE_VM_FINISHED :
AuditLogType.USER_REMOVE_VM_FINISHED_WITH_ILLEGAL_DISKS)
Line 246: : AuditLogType.USER_FAILED_REMOVE_VM;
Done
Line 247: }
Line 248:
Line 249: @Override
Line 250: protected Map<String, Pair<String, String>> getExclusiveLocks() {
Line 273: }
Line 274:
Line 275: @Override
Line 276: protected void endVmCommand() {
Line 277: setCommandShouldBeLogged(false);
Done
Line 278:
Line 279: setSucceeded(true);
Line 280: }
Line 281:
--
To view, visit http://gerrit.ovirt.org/14569
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7a3e497e52471f7255e6c17b59a2e3bf362b8783
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
