Change in ovirt-engine[master]: core: disable command's logging on runningFailed
Arik Hadas has submitted this change and it was merged.
Change subject: core: disable command's logging on runningFailed
......................................................................
core: disable command's logging on runningFailed
After calling RunVmCommandBase#runningFailed the automatic logging
should be disabled since the 'log()' method is called directly by this
method. A call to 'setCommandShouldBeLogged(false)' was added inside
RunVmCommandBase#runningFailed.
It is safe since this method is called on rerun flows, where the
automatic logging is not invoked anyway, and from one place as part of
the execution phase where the automatic logging is already disabled.
Change-Id: Ied9130d2e8dacb94924da220e8ba1a1f1204e681
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java
2 files changed, 1 insertion(+), 1 deletion(-)
Approvals:
Arik Hadas: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/25574
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ied9130d2e8dacb94924da220e8ba1a1f1204e681
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server