Yaniv Bronhaim has posted comments on this change.
Change subject: core: Fixes logging in SshSoftFencingCommand
......................................................................
Patch Set 4: I would prefer that you didn't submit this
(1 inline comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SshSoftFencingCommand.java
Line 81: cmdErr);
Line 82: log.infoFormat("SSH Soft Fencing command executed on host
{0}", getVds().getHostName());
Line 83: } catch (Exception ex) {
Line 84: log.errorFormat("SSH Soft Fencing command failed on host
{0}: {1}\nStdout: {2}\nStderr: {3}\nStacktrace: {4}",
Line 85: getVds().getHostName(), ex.getMessage(),
getCommandOutput(cmdOut), getCommandOutput(cmdErr), ex);
instead of adding a function you can initialize cmdOut and cmdErr with empty
string instead of null...
Line 86: result = false;
Line 87: } finally {
Line 88: closeSshConnection(sshClient);
Line 89: }
--
To view, visit http://gerrit.ovirt.org/16619
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6ea12003bd44f6500e3878176e98debd102da056
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
