Furthur refactoring of the paramiko and ssh host classes. This covers a few
different fixes:
- it pulls the verify & repair code out of SSHHost and into AbstractSSHHost
- it moves all the __init__ code into _initialize methods; there's certain pieces of code (like self.start_loggers()) that always needs to run after all the other initialization, so it's easier to enforce this kind of ordering if we put the core initialization in a separate method and then just have one __init__ method in Host that enforces the ordering
- fixes up ParamikoHost to properly handle forking, since after a fork() one of the processes needs to switch to a new Transport object (both processes can't use the same connection)
Risk: High
Visibility: ParamikoHost now supports fork() properly, provides the same verify & repair as SSHHost, and most of the host __init__ code has migrated into _initialize methods
Signed-off-by: John Admanski <jadmanski@google.com>
git-svn-id: http://test.kernel.org/svn/autotest/trunk@2318 592f7852-d20e-0410-864c-8624ca9c26a4
diff --git a/server/hosts/serial.py b/server/hosts/serial.py
index f654793..e28ad4a 100644
--- a/server/hosts/serial.py
+++ b/server/hosts/serial.py
@@ -8,9 +8,9 @@
class SerialHost(site_host.SiteHost):
DEFAULT_REBOOT_TIMEOUT = site_host.SiteHost.DEFAULT_REBOOT_TIMEOUT
- def __init__(self, conmux_server=None, conmux_attach=None,
- console_log="console.log", *args, **dargs):
- super(SerialHost, self).__init__(*args, **dargs)
+ def _initialize(self, conmux_server=None, conmux_attach=None,
+ console_log="console.log", *args, **dargs):
+ super(SerialHost, self)._initialize(*args, **dargs)
self.__logger = None
self.__console_log = console_log