Oved Ourfali has submitted this change and it was merged.
Change subject: host-deploy: flush both stdout, stderr at ssh execute
......................................................................
host-deploy: flush both stdout, stderr at ssh execute
although until unclear why it is needed.
1. flush before java attempt to close using try-with-resources.
2. flush also stderr.
3. perform at one location.
Related-To:
https://bugzilla.redhat.com/show_bug.cgi?id=1084911
Change-Id: I14424723f2ef6abff6f3abdc1072e5ac621d4e67
Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com>
---
M
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ssh/SSHClient.java
M
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ssh/SSHDialog.java
2 files changed, 10 insertions(+), 12 deletions(-)
Approvals:
Alon Bar-Lev: Verified
Eli Mesika: Looks good to me, but someone else must approve
Yair Zaslavsky: Looks good to me, but someone else must approve
Oved Ourfali: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/26506
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I14424723f2ef6abff6f3abdc1072e5ac621d4e67
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server