Change in ovirt-engine[ovirt-engine-4.0]: core: ChildCommandsCallbackBase - setSucceeded(false)

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: core: ChildCommandsCallbackBase - setSucceeded(false)
......................................................................


core: ChildCommandsCallbackBase - setSucceeded(false)

When the command entity is persisted, its return value is persisted as well.
If the command execute() succeeded, getReturnValue().getSucceeded() will return
true.

The command object is built from the persisted command entity, which means
that the return value will be initiated to the persisted return value, when calling
endAction() having the return value returning true value when checking
getSucceeded() might lead to wrong results.

This patch is simple and sets the succeeded flag to false to avoid that issue.

Change-Id: I39fd64001b0ef5354853aa3e6433d478f8dbb74a
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1339536
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChildCommandsCallbackBase.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Liron Aravot: Verified



-- 
To view, visit https://gerrit.ovirt.org/58020
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I39fd64001b0ef5354853aa3e6433d478f8dbb74a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>