Code Review
/
yardstick.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Bugfix: load_images cleanup, shellcheck fixes
[yardstick.git]
/
yardstick
/
ssh.py
diff --git
a/yardstick/ssh.py
b/yardstick/ssh.py
index
927ca94
..
cfbc3ca
100644
(file)
--- a/
yardstick/ssh.py
+++ b/
yardstick/ssh.py
@@
-25,7
+25,7
@@
Execute command and get output:
status, stdout, stderr = ssh.execute("ps ax")
if status:
raise Exception("Command failed with non-zero status.")
status, stdout, stderr = ssh.execute("ps ax")
if status:
raise Exception("Command failed with non-zero status.")
- print
stdout.splitlines(
)
+ print
(stdout.splitlines()
)
Execute command with huge output:
Execute command with huge output:
@@
-62,6
+62,7
@@
Eventlet:
sshclient = eventlet.import_patched("yardstick.ssh")
"""
sshclient = eventlet.import_patched("yardstick.ssh")
"""
+from __future__ import absolute_import
import os
import select
import socket
import os
import select
import socket
@@
-70,6
+71,7
@@
import re
import logging
import paramiko
import logging
import paramiko
+from oslo_utils import encodeutils
from scp import SCPClient
import six
from scp import SCPClient
import six
@@
-199,7
+201,8
@@
class SSH(object):
session.exec_command(cmd)
start_time = time.time()
session.exec_command(cmd)
start_time = time.time()
- data_to_send = ""
+ # encode on transmit, decode on receive
+ data_to_send = encodeutils.safe_encode("", incoming='utf-8')
stderr_data = None
# If we have data to be sent to stdin then `select' should also
stderr_data = None
# If we have data to be sent to stdin then `select' should also
@@
-214,14
+217,15
@@
class SSH(object):
r, w, e = select.select([session], writes, [session], 1)
if session.recv_ready():
r, w, e = select.select([session], writes, [session], 1)
if session.recv_ready():
- data =
session.recv(4096
)
+ data =
encodeutils.safe_decode(session.recv(4096), 'utf-8'
)
self.log.debug("stdout: %r", data)
if stdout is not None:
stdout.write(data)
continue
if session.recv_stderr_ready():
self.log.debug("stdout: %r", data)
if stdout is not None:
stdout.write(data)
continue
if session.recv_stderr_ready():
- stderr_data = session.recv_stderr(4096)
+ stderr_data = encodeutils.safe_decode(
+ session.recv_stderr(4096), 'utf-8')
self.log.debug("stderr: %r", stderr_data)
if stderr is not None:
stderr.write(stderr_data)
self.log.debug("stderr: %r", stderr_data)
if stderr is not None:
stderr.write(stderr_data)
@@
-230,7
+234,11
@@
class SSH(object):
if session.send_ready():
if stdin is not None and not stdin.closed:
if not data_to_send:
if session.send_ready():
if stdin is not None and not stdin.closed:
if not data_to_send:
- data_to_send = stdin.read(4096)
+ stdin_txt = stdin.read(4096)
+ if stdin_txt is None:
+ stdin_txt = ''
+ data_to_send = encodeutils.safe_encode(
+ stdin_txt, incoming='utf-8')
if not data_to_send:
# we may need to keep stdin open
if not keep_stdin_open:
if not data_to_send:
# we may need to keep stdin open
if not keep_stdin_open: