bonus points, catch remoto issue #3

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

diff --git a/ceph_deploy/osd.py b/ceph_deploy/osd.py
index 2aea14e4f0333fd8f8e76bb46b6a8f75049c06ef..7fc405d83a3714144f7255084cdc064901819267 100644 (file)
--- a/ceph_deploy/osd.py
+++ b/ceph_deploy/osd.py
@@ -124,10 +124,17 @@ def osd_status_check(conn, cluster):
         '--format=json',
     ]
 
-    out, err, code = process.check(
-        conn,
-        command,
-    )
+    try:
+        out, err, code = process.check(
+            conn,
+            command,
+        )
+    except TypeError:
+        # XXX This is a bug in remoto. If the other end disconnects with a timeout
+        # it will return a None, and here we are expecting a 3 item tuple, not a None
+        # so it will break with a TypeError. Once remoto fixes this, we no longer need
+        # this try/except.
+        return {}
 
     try:
         loaded_json = json.loads(''.join(out))