[node-patches] Change in ovirt-node[master]: Set UTF-8 encoding on process.pipe as well for Py 2.6 compat

fabiand at fedoraproject.org fabiand at fedoraproject.org
Tue Oct 1 12:17:23 UTC 2013


Fabian Deutsch has submitted this change and it was merged.

Change subject: Set UTF-8 encoding on process.pipe as well for Py 2.6 compat
......................................................................


Set UTF-8 encoding on process.pipe as well for Py 2.6 compat

Previously, we only explicitly returned unicode-encoded output for
process.check_output. Do the same for process.pipe so lsblk on
the diagnostic page doesn't crash when it sees UTF-8 characters.

Change-Id: I0801bcd989da89749272d1e0f16dd4d17129c97e
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/utils/process.py
1 file changed, 12 insertions(+), 4 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/19626
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0801bcd989da89749272d1e0f16dd4d17129c97e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list