Liron Aravot has posted comments on this change. Change subject: core: change return value of AddDisk ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/36425/2/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java: Line 452: Line 453: if (tmpRetValue.getActionReturnValue() != null) { Line 454: DiskImage diskImage = (DiskImage) tmpRetValue.getActionReturnValue(); Line 455: addDiskPermissions(diskImage); Line 456: getReturnValue().setActionReturnValue(isExecutedAsChildCommand() ? diskImage : diskImage.getId()); I think that having different return values is kind of confusing, additionally- we also have the case of LUN disks which return the id of the created disk on any case (internal or not). I think that we should either change the return value completely or to continue as is. Line 457: } Line 458: getReturnValue().setFault(tmpRetValue.getFault()); Line 459: setSucceeded(tmpRetValue.getSucceeded()); Line 460: } -- To view, visit http://gerrit.ovirt.org/36425 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6a41d57bfb1bf48f0a2c6a3703e612b27da50db8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Liron Aravot <lara...@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
