diff options
author | Salvatore Orlando <salvatore.orlando@eu.citrix.com> | 2011-02-10 12:22:54 +0000 |
---|---|---|
committer | Salvatore Orlando <salvatore.orlando@eu.citrix.com> | 2011-02-10 12:22:54 +0000 |
commit | 648a26d2980b54edafbf0c924a96e6be0dfdb794 (patch) | |
tree | 5260d87098a5b34552c049a1c3a7232c7d5ec37e /nova/utils.py | |
parent | f6ec9568561dd430bd772f171f5bbddd0bd038c6 (diff) | |
parent | 5e66cf492f150bfbd01e5983d876192c5b158343 (diff) | |
download | nova-648a26d2980b54edafbf0c924a96e6be0dfdb794.tar.gz nova-648a26d2980b54edafbf0c924a96e6be0dfdb794.tar.xz nova-648a26d2980b54edafbf0c924a96e6be0dfdb794.zip |
Merge from trunk
Diffstat (limited to 'nova/utils.py')
-rw-r--r-- | nova/utils.py | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/nova/utils.py b/nova/utils.py index 5f5225289..8d7ff1f64 100644 --- a/nova/utils.py +++ b/nova/utils.py @@ -152,6 +152,42 @@ def execute(cmd, process_input=None, addl_env=None, check_exit_code=True): return result +def ssh_execute(ssh, cmd, process_input=None, + addl_env=None, check_exit_code=True): + LOG.debug(_("Running cmd (SSH): %s"), cmd) + if addl_env: + raise exception.Error("Environment not supported over SSH") + + if process_input: + # This is (probably) fixable if we need it... + raise exception.Error("process_input not supported over SSH") + + stdin_stream, stdout_stream, stderr_stream = ssh.exec_command(cmd) + channel = stdout_stream.channel + + #stdin.write('process_input would go here') + #stdin.flush() + + # NOTE(justinsb): This seems suspicious... + # ...other SSH clients have buffering issues with this approach + stdout = stdout_stream.read() + stderr = stderr_stream.read() + stdin_stream.close() + + exit_status = channel.recv_exit_status() + + # exit_status == -1 if no exit code was returned + if exit_status != -1: + LOG.debug(_("Result was %s") % exit_status) + if check_exit_code and exit_status != 0: + raise exception.ProcessExecutionError(exit_code=exit_status, + stdout=stdout, + stderr=stderr, + cmd=cmd) + + return (stdout, stderr) + + def abspath(s): return os.path.join(os.path.dirname(__file__), s) |