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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
