Federico Simoncelli has posted comments on this change.
Change subject: Use the parent parameters in RemoveAllVmImagesCommand
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveAllVmImagesCommand.java
Line 51: for (final DiskImage image : images) {
Line 52: if (mImagesToBeRemoved.contains(image.getImageId())) {
Line 53: RemoveImageParameters tempVar = new
RemoveImageParameters(image.getImageId());
Line 54:
tempVar.setParentCommand(getParameters().getParentCommand());
Line 55:
tempVar.setParentParameters(getParameters().getParentParameters());
The parent parameters were already (correctly) set here.
Line 56: tempVar.setDiskImage(image);
Line 57: tempVar.setEntityId(getParameters().getEntityId());
Line 58:
tempVar.setForceDelete(getParameters().getForceDelete());
Line 59: VdcReturnValueBase vdcReturnValue =
--
To view, visit http://gerrit.ovirt.org/8119
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If4466e2e29bcaf67c44f51c93dc6b98ca916bae9
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
