Merge ~kissiel/checkbox-ng:fix-crash-on-second-session into checkbox-ng:master

[Maciej Kisielewski]

Fix slave crashing when a second session is started without restarting the slave.

See https://bugs.launchpad.net/checkbox-ng/+bug/1874105 for details

[Jonathan Cave (jocave)]

Waiting on potential fixes for problems during report generation before landing this.

[Jonathan Cave (jocave)]

In combination with https://code.launchpad.net/~kissiel/checkbox-ng/+git/checkbox-ng/+merge/383061 this appears to allow for multiple sessions that conclude cleanly. Thanks!

+1

review: Approve

diff --git a/checkbox_ng/launcher/slave.py b/checkbox_ng/launcher/slave.py
index 81b884e..c1b03ea 100644
--- a/checkbox_ng/launcher/slave.py
+++ b/checkbox_ng/launcher/slave.py
@@ -77,6 +77,10 @@ class SessionAssistantSlave(rpyc.Service):
             old_master.close()
             SessionAssistantSlave.controlling_master_conn = conn
 
+    def on_disconnect(self, conn):
+        SessionAssistantSlave.master_blaster = None
+        self.controlling_master_conn = None
+
 
 class RemoteSlave():
     """