J. Richard Barnette | 67ccb87 | 2012-04-19 16:34:56 -0700 | [diff] [blame] | 1 | # Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
Vadim Bendebury | 692341e | 2012-12-18 15:24:38 -0800 | [diff] [blame] | 5 | import httplib, logging, os, socket, subprocess, sys, time, xmlrpclib |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 6 | |
Vic Yang | 6924955 | 2013-05-09 01:58:11 +0800 | [diff] [blame] | 7 | from autotest_lib.client.common_lib import error, utils |
Vadim Bendebury | bb73195 | 2012-12-05 17:30:36 -0800 | [diff] [blame] | 8 | from autotest_lib.server import autotest, test |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 9 | |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 10 | |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 11 | class ServoTest(test.test): |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 12 | """AutoTest test class to serve as a parent class for FAFT tests. |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 13 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 14 | TODO(jrbarnette): This class is a legacy, reflecting |
| 15 | refactoring that has begun but not completed. The long term |
| 16 | plan is to move all function here into FAFT specific classes. |
| 17 | http://crosbug.com/33305. |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 18 | """ |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 19 | version = 2 |
J. Richard Barnette | 67ccb87 | 2012-04-19 16:34:56 -0700 | [diff] [blame] | 20 | |
Vic Yang | 8535e77 | 2013-05-02 09:07:25 +0800 | [diff] [blame] | 21 | _REMOTE_PORT = 9990 |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 22 | _REMOTE_COMMAND = '/usr/local/autotest/cros/faft_client.py' |
| 23 | _REMOTE_COMMAND_SHORT = 'faft_client' |
| 24 | _REMOTE_LOG_FILE = '/tmp/faft_client.log' |
| 25 | _SSH_CONFIG = ('-o StrictHostKeyChecking=no ' |
| 26 | '-o UserKnownHostsFile=/dev/null ') |
J. Richard Barnette | 67ccb87 | 2012-04-19 16:34:56 -0700 | [diff] [blame] | 27 | |
Tom Wai-Hong Tam | 54f4c58 | 2013-07-18 12:05:27 +0800 | [diff] [blame^] | 28 | def initialize(self, host): |
J. Richard Barnette | 67ccb87 | 2012-04-19 16:34:56 -0700 | [diff] [blame] | 29 | """Create a Servo object and install the dependency. |
J. Richard Barnette | 67ccb87 | 2012-04-19 16:34:56 -0700 | [diff] [blame] | 30 | """ |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 31 | self.servo = host.servo |
| 32 | self.faft_client = None |
| 33 | self._client = host |
| 34 | self._ssh_tunnel = None |
| 35 | self._remote_process = None |
Vic Yang | 8535e77 | 2013-05-02 09:07:25 +0800 | [diff] [blame] | 36 | self._local_port = None |
J. Richard Barnette | 67ccb87 | 2012-04-19 16:34:56 -0700 | [diff] [blame] | 37 | |
Chrome Bot | 9a1137d | 2011-07-19 14:35:00 -0700 | [diff] [blame] | 38 | # Initializes dut, may raise AssertionError if pre-defined gpio |
| 39 | # sequence to set GPIO's fail. Autotest does not handle exception |
| 40 | # throwing in initialize and will cause a test to hang. |
| 41 | try: |
| 42 | self.servo.initialize_dut() |
Chris Sosa | 33320a8 | 2011-10-24 14:28:32 -0700 | [diff] [blame] | 43 | except (AssertionError, xmlrpclib.Fault) as e: |
Chrome Bot | 9a1137d | 2011-07-19 14:35:00 -0700 | [diff] [blame] | 44 | raise error.TestFail(e) |
| 45 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 46 | # Install faft_client dependency. |
| 47 | self._autotest_client = autotest.Autotest(self._client) |
| 48 | self._autotest_client.install() |
| 49 | self._launch_client() |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 50 | |
J. Richard Barnette | 134ec2c | 2012-04-25 12:59:37 -0700 | [diff] [blame] | 51 | def _ping_test(self, hostname, timeout=5): |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 52 | """Verify whether a host responds to a ping. |
| 53 | |
| 54 | Args: |
| 55 | hostname: Hostname to ping. |
| 56 | timeout: Time in seconds to wait for a response. |
| 57 | """ |
Tom Wai-Hong Tam | 4a257e5 | 2011-11-12 08:36:22 +0800 | [diff] [blame] | 58 | with open(os.devnull, 'w') as fnull: |
| 59 | return subprocess.call( |
| 60 | ['ping', '-c', '1', '-W', str(timeout), hostname], |
| 61 | stdout=fnull, stderr=fnull) == 0 |
Craig Harrison | 2b6c6fc | 2011-06-23 10:34:02 -0700 | [diff] [blame] | 62 | |
ctchang | 74816ac | 2012-08-31 11:12:43 +0800 | [diff] [blame] | 63 | def _sshd_test(self, hostname, timeout=5): |
| 64 | """Verify whether sshd is running in host. |
| 65 | |
| 66 | Args: |
| 67 | hostname: Hostname to verify. |
| 68 | timeout: Time in seconds to wait for a response. |
| 69 | """ |
| 70 | try: |
| 71 | sock = socket.create_connection((hostname, 22), timeout=timeout) |
| 72 | sock.close() |
| 73 | return True |
Tom Wai-Hong Tam | 711a7aa | 2012-09-18 10:30:43 +0800 | [diff] [blame] | 74 | except socket.timeout: |
| 75 | return False |
ctchang | 74816ac | 2012-08-31 11:12:43 +0800 | [diff] [blame] | 76 | except socket.error: |
Tom Wai-Hong Tam | 711a7aa | 2012-09-18 10:30:43 +0800 | [diff] [blame] | 77 | time.sleep(timeout) |
ctchang | 74816ac | 2012-08-31 11:12:43 +0800 | [diff] [blame] | 78 | return False |
| 79 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 80 | def _launch_client(self): |
Tom Wai-Hong Tam | e97cb4a | 2012-11-22 09:52:46 +0800 | [diff] [blame] | 81 | """Launch a remote XML RPC connection on client with retrials. |
Tom Wai-Hong Tam | e97cb4a | 2012-11-22 09:52:46 +0800 | [diff] [blame] | 82 | """ |
| 83 | retry = 3 |
Tom Wai-Hong Tam | b3db3b2 | 2012-12-04 11:59:54 +0800 | [diff] [blame] | 84 | while retry: |
| 85 | try: |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 86 | self._launch_client_once() |
Tom Wai-Hong Tam | b3db3b2 | 2012-12-04 11:59:54 +0800 | [diff] [blame] | 87 | break |
| 88 | except AssertionError: |
Tom Wai-Hong Tam | e97cb4a | 2012-11-22 09:52:46 +0800 | [diff] [blame] | 89 | retry -= 1 |
Tom Wai-Hong Tam | b3db3b2 | 2012-12-04 11:59:54 +0800 | [diff] [blame] | 90 | if retry: |
| 91 | logging.info('Retry again...') |
| 92 | time.sleep(5) |
| 93 | else: |
| 94 | raise |
Tom Wai-Hong Tam | e97cb4a | 2012-11-22 09:52:46 +0800 | [diff] [blame] | 95 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 96 | def _launch_client_once(self): |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 97 | """Launch a remote process on client and set up an xmlrpc connection. |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 98 | """ |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 99 | if self._ssh_tunnel: |
| 100 | self._ssh_tunnel.terminate() |
| 101 | self._ssh_tunnel = None |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 102 | |
| 103 | # Launch RPC server remotely. |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 104 | self._kill_remote_process() |
| 105 | self._launch_ssh_tunnel() |
Vadim Bendebury | 692341e | 2012-12-18 15:24:38 -0800 | [diff] [blame] | 106 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 107 | logging.info('Client command: %s', self._REMOTE_COMMAND) |
| 108 | logging.info("Logging to %s", self._REMOTE_LOG_FILE) |
Vic Yang | a349237 | 2013-05-16 16:11:19 +0800 | [diff] [blame] | 109 | full_cmd = ['ssh -n %s root@%s \'%s &> %s\'' % ( |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 110 | self._SSH_CONFIG, self._client.ip, |
| 111 | self._REMOTE_COMMAND, self._REMOTE_LOG_FILE)] |
Vadim Bendebury | 89ec24e | 2012-12-17 12:54:18 -0800 | [diff] [blame] | 112 | logging.info('Starting process %s', ' '.join(full_cmd)) |
Vic Yang | a349237 | 2013-05-16 16:11:19 +0800 | [diff] [blame] | 113 | self._remote_process = subprocess.Popen(full_cmd, shell=True, |
| 114 | stdout=subprocess.PIPE, |
| 115 | stderr=subprocess.PIPE) |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 116 | |
| 117 | # Connect to RPC object. |
| 118 | logging.info('Connecting to client RPC server...') |
Vic Yang | 8535e77 | 2013-05-02 09:07:25 +0800 | [diff] [blame] | 119 | remote_url = 'http://localhost:%s' % self._local_port |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 120 | self.faft_client = xmlrpclib.ServerProxy(remote_url, allow_none=True) |
Vic Yang | 3a7cf60 | 2012-11-07 17:28:39 +0800 | [diff] [blame] | 121 | logging.info('Server proxy: %s', remote_url) |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 122 | |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 123 | # Poll for client RPC server to come online. |
Vic Yang | 6c1dc15 | 2012-09-13 14:56:11 +0800 | [diff] [blame] | 124 | timeout = 20 |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 125 | succeed = False |
Vic Yang | 3a7cf60 | 2012-11-07 17:28:39 +0800 | [diff] [blame] | 126 | rpc_error = None |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 127 | while timeout > 0 and not succeed: |
Vic Yang | 6c1dc15 | 2012-09-13 14:56:11 +0800 | [diff] [blame] | 128 | time.sleep(1) |
Vic Yang | a349237 | 2013-05-16 16:11:19 +0800 | [diff] [blame] | 129 | if self._remote_process.poll() is not None: |
| 130 | # The SSH process is gone. Log stderr. |
| 131 | logging.error('Remote process died!') |
| 132 | sout, serr = self._remote_process.communicate() |
| 133 | logging.error('Stdout: %s', sout) |
| 134 | logging.error('Stderr: %s', serr) |
| 135 | break |
| 136 | |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 137 | try: |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 138 | self.faft_client.system.is_available() |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 139 | succeed = True |
Vadim Bendebury | 692341e | 2012-12-18 15:24:38 -0800 | [diff] [blame] | 140 | except (socket.error, |
| 141 | xmlrpclib.ProtocolError, |
| 142 | httplib.BadStatusLine) as e: |
Yusuf Mohsinally | f3cb764 | 2013-04-15 14:20:56 -0700 | [diff] [blame] | 143 | logging.info('caught: %s %s, tries left: %s', |
| 144 | repr(e), str(e), timeout) |
Tom Wai-Hong Tam | ac35f08 | 2012-11-06 15:00:35 +0800 | [diff] [blame] | 145 | # The client RPC server may not come online fast enough. Retry. |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 146 | timeout -= 1 |
Vic Yang | 3a7cf60 | 2012-11-07 17:28:39 +0800 | [diff] [blame] | 147 | rpc_error = e |
Vadim Bendebury | 692341e | 2012-12-18 15:24:38 -0800 | [diff] [blame] | 148 | except: |
| 149 | logging.error('Unexpected error: %s', sys.exc_info()[0]) |
| 150 | raise |
Vic Yang | f8fd454 | 2012-08-01 11:37:46 +0800 | [diff] [blame] | 151 | |
| 152 | if not succeed: |
Vic Yang | 3a7cf60 | 2012-11-07 17:28:39 +0800 | [diff] [blame] | 153 | if isinstance(rpc_error, xmlrpclib.ProtocolError): |
Tom Wai-Hong Tam | ac35f08 | 2012-11-06 15:00:35 +0800 | [diff] [blame] | 154 | logging.info("A protocol error occurred") |
Vic Yang | 3a7cf60 | 2012-11-07 17:28:39 +0800 | [diff] [blame] | 155 | logging.info("URL: %s", rpc_error.url) |
| 156 | logging.info("HTTP/HTTPS headers: %s", rpc_error.headers) |
| 157 | logging.info("Error code: %d", rpc_error.errcode) |
| 158 | logging.info("Error message: %s", rpc_error.errmsg) |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 159 | p = subprocess.Popen([ |
| 160 | 'ssh -n -q %s root@%s \'cat %s\'' % (self._SSH_CONFIG, |
| 161 | self._client.ip, self._REMOTE_LOG_FILE)], shell=True, |
| 162 | stdout=subprocess.PIPE) |
| 163 | logging.info('Log of running remote %s:', |
| 164 | self._REMOTE_COMMAND_SHORT) |
| 165 | logging.info(p.communicate()[0]) |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 166 | assert succeed, 'Timed out connecting to client RPC server.' |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 167 | |
Vic Yang | 8bbc1c3 | 2012-09-13 11:47:44 +0800 | [diff] [blame] | 168 | def wait_for_client(self, install_deps=False, timeout=100): |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 169 | """Wait for the client to come back online. |
| 170 | |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 171 | New remote processes will be launched if their used flags are enabled. |
Tom Wai-Hong Tam | 7c17ff2 | 2011-10-26 09:44:09 +0800 | [diff] [blame] | 172 | |
Yusuf Mohsinally | f3cb764 | 2013-04-15 14:20:56 -0700 | [diff] [blame] | 173 | @param install_deps: If True, install Autotest dependency when ready. |
| 174 | @param timeout: Time in seconds to wait for the client SSH daemon to |
| 175 | come up. |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 176 | """ |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 177 | # Ensure old ssh connections are terminated. |
| 178 | self._terminate_all_ssh() |
| 179 | # Wait for the client to come up. |
Vic Yang | 6c1dc15 | 2012-09-13 14:56:11 +0800 | [diff] [blame] | 180 | while timeout > 0 and not self._sshd_test(self._client.ip, timeout=2): |
| 181 | timeout -= 2 |
Vadim Bendebury | 69f047c | 2012-10-11 15:20:31 -0700 | [diff] [blame] | 182 | assert (timeout > 0), 'Timed out waiting for client to reboot.' |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 183 | logging.info('Server: Client machine is up.') |
| 184 | # Relaunch remote clients. |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 185 | if install_deps: |
| 186 | self._autotest_client.install() |
| 187 | self._launch_client() |
| 188 | logging.info('Server: Relaunched remote %s.', 'faft') |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 189 | |
Tom Wai-Hong Tam | a9b225b | 2013-05-03 10:35:10 +0800 | [diff] [blame] | 190 | def wait_for_client_offline(self, timeout=60, orig_boot_id=None): |
Tom Wai-Hong Tam | a70f0fe | 2011-09-02 18:28:47 +0800 | [diff] [blame] | 191 | """Wait for the client to come offline. |
| 192 | |
Yusuf Mohsinally | f3cb764 | 2013-04-15 14:20:56 -0700 | [diff] [blame] | 193 | @param timeout: Time in seconds to wait the client to come offline. |
Tom Wai-Hong Tam | 1263606 | 2013-04-09 17:14:15 +0800 | [diff] [blame] | 194 | @param orig_boot_id: A string containing the original boot id. |
Tom Wai-Hong Tam | a70f0fe | 2011-09-02 18:28:47 +0800 | [diff] [blame] | 195 | """ |
| 196 | # Wait for the client to come offline. |
J. Richard Barnette | 134ec2c | 2012-04-25 12:59:37 -0700 | [diff] [blame] | 197 | while timeout > 0 and self._ping_test(self._client.ip, timeout=1): |
Vic Yang | cf5d6fc | 2012-09-14 15:22:44 +0800 | [diff] [blame] | 198 | time.sleep(1) |
Tom Wai-Hong Tam | a70f0fe | 2011-09-02 18:28:47 +0800 | [diff] [blame] | 199 | timeout -= 1 |
Tom Wai-Hong Tam | 1263606 | 2013-04-09 17:14:15 +0800 | [diff] [blame] | 200 | |
| 201 | # As get_boot_id() requires DUT online. So we move the comparison here. |
| 202 | if timeout == 0 and orig_boot_id: |
| 203 | if self._client.get_boot_id() != orig_boot_id: |
| 204 | logging.warn('Reboot done very quickly.') |
| 205 | return |
| 206 | |
Tom Wai-Hong Tam | a70f0fe | 2011-09-02 18:28:47 +0800 | [diff] [blame] | 207 | assert timeout, 'Timed out waiting for client offline.' |
| 208 | logging.info('Server: Client machine is offline.') |
| 209 | |
Vic Yang | 4e0d1f7 | 2012-05-24 15:11:11 +0800 | [diff] [blame] | 210 | def kill_remote(self): |
| 211 | """Call remote cleanup and kill ssh.""" |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 212 | if self._remote_process and self._remote_process.poll() is None: |
| 213 | try: |
| 214 | self.faft_client.cleanup() |
| 215 | logging.info('Cleanup succeeded.') |
| 216 | except xmlrpclib.ProtocolError, e: |
| 217 | logging.info('Cleanup returned protocol error: ' + str(e)) |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 218 | self._terminate_all_ssh() |
| 219 | |
Vic Yang | 4e0d1f7 | 2012-05-24 15:11:11 +0800 | [diff] [blame] | 220 | def cleanup(self): |
| 221 | """Delete the Servo object, call remote cleanup, and kill ssh.""" |
Vic Yang | 4e0d1f7 | 2012-05-24 15:11:11 +0800 | [diff] [blame] | 222 | self.kill_remote() |
| 223 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 224 | def _launch_ssh_tunnel(self): |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 225 | """Establish an ssh tunnel for connecting to the remote RPC server. |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 226 | """ |
Vic Yang | 8535e77 | 2013-05-02 09:07:25 +0800 | [diff] [blame] | 227 | if self._local_port is None: |
Vic Yang | 6924955 | 2013-05-09 01:58:11 +0800 | [diff] [blame] | 228 | self._local_port = utils.get_unused_port() |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 229 | if not self._ssh_tunnel or self._ssh_tunnel.poll() is not None: |
| 230 | self._ssh_tunnel = subprocess.Popen([ |
Tom Wai-Hong Tam | 4a257e5 | 2011-11-12 08:36:22 +0800 | [diff] [blame] | 231 | 'ssh -N -n -q %s -L %s:localhost:%s root@%s' % |
Vic Yang | 8535e77 | 2013-05-02 09:07:25 +0800 | [diff] [blame] | 232 | (self._SSH_CONFIG, self._local_port, self._REMOTE_PORT, |
Tom Wai-Hong Tam | e2c6612 | 2011-10-25 17:21:35 +0800 | [diff] [blame] | 233 | self._client.ip)], shell=True) |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 234 | assert self._ssh_tunnel.poll() is None, \ |
Vic Yang | 8535e77 | 2013-05-02 09:07:25 +0800 | [diff] [blame] | 235 | 'The SSH tunnel on port %d is not up.' % self._local_port |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 236 | |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 237 | def _kill_remote_process(self): |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 238 | """Ensure the remote process and local ssh process are terminated. |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 239 | """ |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 240 | kill_cmd = 'pkill -f %s' % self._REMOTE_COMMAND_SHORT |
Tom Wai-Hong Tam | 4a257e5 | 2011-11-12 08:36:22 +0800 | [diff] [blame] | 241 | subprocess.call(['ssh -n -q %s root@%s \'%s\'' % |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 242 | (self._SSH_CONFIG, self._client.ip, kill_cmd)], |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 243 | shell=True) |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 244 | if self._remote_process and self._remote_process.poll() is None: |
| 245 | self._remote_process.terminate() |
Craig Harrison | 9194455 | 2011-08-04 14:09:55 -0700 | [diff] [blame] | 246 | |
| 247 | def _terminate_all_ssh(self): |
Tom Wai-Hong Tam | bea57b3 | 2011-09-02 18:27:47 +0800 | [diff] [blame] | 248 | """Terminate all ssh connections associated with remote processes.""" |
J. Richard Barnette | 33aec9f | 2013-02-01 16:38:41 -0800 | [diff] [blame] | 249 | if self._ssh_tunnel and self._ssh_tunnel.poll() is None: |
| 250 | self._ssh_tunnel.terminate() |
| 251 | self._kill_remote_process() |
| 252 | self._ssh_tunnel = None |