Change in ovirt-engine[master]: core: SpmStopOnIrs - set result

laravot at redhat.com laravot at redhat.com
Mon Aug 18 10:58:04 UTC 2014


Liron Aravot has submitted this change and it was merged.

Change subject: core: SpmStopOnIrs - set result
......................................................................


core: SpmStopOnIrs - set result

When executing SpmStopOnIrs we should set the return value to be
as the return value of the command that's being executed, otherwise the
caller can't tell what actually happend.

Change-Id: I5eb2a4c788ef5cf571a8de150355c85e3a9ae77a
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1130222
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/SpmStopOnIrsVDSCommand.java
1 file changed, 6 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve
  Liron Aravot: Verified



-- 
To view, visit http://gerrit.ovirt.org/31563
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5eb2a4c788ef5cf571a8de150355c85e3a9ae77a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list