Moti Asayag has submitted this change and it was merged. Change subject: core: Change VDSBrokerFrontend return value of remove ......................................................................
core: Change VDSBrokerFrontend return value of remove The patch changes the return value of VDSBrokerFrontend.RemoveAsyncRunningCommand from void to the command which being removed. The purpose of the change is to allow invoking the reportCompleted() method of the removed command in order to verify the job or step describe the command are properly ended. Change-Id: I22c1f558be62e86f6fcd72ffdaf033a5ba163b2e --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VDSBrokerFrontendImpl.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RunVmCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/interfaces/VDSBrokerFrontend.java 3 files changed, 10 insertions(+), 10 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1297 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I22c1f558be62e86f6fcd72ffdaf033a5ba163b2e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <[email protected]> Gerrit-Reviewer: Michael Kublin <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
