Changeset 1598

Timestamp:

Nov 10, 2008, 3:20:05 PM (16 years ago)

Author:

broder

Message:

Don't fail to execute remctls if a single server is down

Location:

trunk/packages/invirt-remote-server

Files:

• debian/changelog (modified) (1 diff)
• python/remote/bcast.py (modified) (1 diff)

trunk/packages/invirt-remote-server/debian/changelog

@@ -2 +2 @@
 
   * Include invirt.remote Python module
+  * Ignore erroneous responses from a server if it's because the server is
+    down
 
- -- Evan Broder <broder@mit.edu>  Mon, 10 Nov 2008 15:15:13 -0500
+ -- Evan Broder <broder@mit.edu>  Mon, 10 Nov 2008 15:19:28 -0500
 
 invirt-remote-server (0.0.14) unstable; urgency=low

trunk/packages/invirt-remote-server/python/remote/bcast.py

@@ -10 +10 @@
     """
     pipes = [(host,
-              Popen(['remctl', host, 'remote', 'web', cmd], stdout=PIPE))
+              Popen(['remctl', host, 'remote', 'web', cmd], stdout=PIPE, stderr=PIPE))
              for host in hosts]
-    outputs = [(s, p.communicate()[0]) for (s, p) in pipes]
+    outputs = dict((s, p.communicate()) for (s, p) in pipes)
     for (s, p) in pipes:
         if p.returncode != 0:
-            raise RuntimeError("remctl to host %s returned non-zero exit status %d"
-                               % (s, p.returncode))
-    return [(s, yaml.load(o, yaml.CSafeLoader)) for (s, o) in outputs]
+            if outputs[s][1].startswith('remctl: cannot connect to %s' % s):
+                del outputs[s]
+            else:
+                raise RuntimeError("remctl to host %s returned non-zero exit status %d"
+                                   % (s, p.returncode))
+    return [(s, yaml.load(o[1], yaml.CSafeLoader)) for (s, o) in outputs.iteritems()]