WIP: exit status changes/hacks for fakessh

mitogen/fakessh.py

@@ -163,12 +163,39 @@ class Process(object):
         self.stdin = mitogen.core.Sender(dest, arg[1])
         self.router.broker.start_receive(self.pump)
 
+        if self.proc:
+            pmon = mitogen.master.ProcessMonitor.instance()
+            pmon.add(self.proc.pid, self._on_proc_exit)
+
+    exitted = False
+
     def _on_exit(self, msg, arg):
-        LOG.debug('on_exit: proc = %r', self.proc)
+        LOG.debug('on_exit: proc = %r, arg = %r', self.proc, arg)
         if self.proc:
-            self.proc.terminate()
+            try:
+                self.proc.terminate()
+            except OSError:
+                pass
         else:
-            self.router.broker.shutdown()
+            #self.router.broker.shutdown()
+            msg = (
+                mitogen.core.Message.pickled(
+                    ('sys', None, 'exit', (arg or 0,), {}),
+                    handle=mitogen.core.CALL_FUNCTION,
+                    src_id=mitogen.parent_ids[0],
+                    dst_id=mitogen.context_id,
+                )
+            )
+            print 'HAHA'
+            print msg
+            self.router.route(
+                mitogen.core.Message.pickled(
+                    ('sys', None, 'exit', (arg or 0,), {}),
+                    handle=mitogen.core.CALL_FUNCTION,
+                    src_id=mitogen.parent_ids[0],
+                    dst_id=mitogen.context_id,
+                )
+            )
 
     def _on_pump_receive(self, s):
         IOLOG.info('%r._on_pump_receive(len %d)', self, len(s))
@@ -205,8 +232,10 @@ def _start_slave(src_id, cmdline, router):
     """
     LOG.debug('_start_slave(%r, %r)', router, cmdline)
 
-    proc = subprocess.Popen(
+    # Must be constructed from main thread due to signal use.
-        cmdline,
+    pmon = mitogen.master.ProcessMonitor.instance()
+
+    proc = subprocess.Popen(cmdline,
         # SSH server always uses user's shell.
         shell=True,
         # SSH server always executes new commands in the user's HOME.

tests/fakessh_test.py

@@ -9,6 +9,19 @@ import mitogen.fakessh
 import testlib
 
 
+import logging
+logging.basicConfig(level=logging.DEBUG)
+class ExitStatusTest(testlib.RouterMixin, unittest2.TestCase):
+    # Ensure child's exit status is propagated to the parent.
+    def test_exit_0(self):
+        local = self.router.local()
+
+        exit_status = mitogen.fakessh.run(local, self.router, [
+            'bash', '-c', 'exec ssh foo exit 0'
+        ])
+        self.assertEquals(0, exit_status)
+
+
 class RsyncTest(testlib.DockerMixin, unittest2.TestCase):
     def test_rsync_from_master(self):
         context = self.docker_ssh_any()
@@ -17,7 +30,7 @@ class RsyncTest(testlib.DockerMixin, unittest2.TestCase):
             context.call(shutil.rmtree, '/tmp/data')
 
         return_code = mitogen.fakessh.run(context, self.router, [
-            'rsync', '--progress', '-vvva',
+            'rsync', '-a',
             testlib.data_path('.'), 'target:/tmp/data'
         ])
 
@@ -49,7 +62,7 @@ class RsyncTest(testlib.DockerMixin, unittest2.TestCase):
             webapp_acct.call(shutil.rmtree, dest_path)
 
         return_code = pubkey_acct.call(mitogen.fakessh.run, webapp_acct, args=[
-            'rsync', '--progress', '-vvva', '.ssh/', 'target:' + dest_path
+            'rsync', '-a', '.ssh/', 'target:' + dest_path
         ])
 
         self.assertEqual(return_code, 0)