From: Greg Price <price@mit.edu>
Date: Sat, 31 Jan 2009 08:14:30 +0000 (-0500)
Subject: invirt.remote.bcast: provide stderr text when remctl fails
X-Git-Tag: 0.3.1~1
X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-remote.git/commitdiff_plain/56034cb2436162eda87cd6e9228ed8751cbc87c1

invirt.remote.bcast: provide stderr text when remctl fails

svn path=/trunk/packages/invirt-remote/; revision=2045
---

diff --git a/debian/changelog b/debian/changelog
index 9f0c1d7..86d1a5c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+invirt-remote (0.3.1) unstable; urgency=low
+
+  * invirt.remote.bcast: provide stderr text when remctl fails
+
+ -- Greg Price <price@mit.edu>  Sat, 31 Jan 2009 02:47:28 -0500
+
 invirt-remote (0.3.0) unstable; urgency=low
 
   * Instead of immediately deleting LVs, overwrite them with zeros to
diff --git a/python/remote/bcast.py b/python/remote/bcast.py
index 6a00b23..4544af7 100644
--- a/python/remote/bcast.py
+++ b/python/remote/bcast.py
@@ -17,6 +17,6 @@ def bcast(cmd, hosts = [h.hostname for h in config.hosts]):
             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))
+                raise RuntimeError("remctl to host %s returned non-zero exit status %d; stderr:\n%s"
+                                   % (s, p.returncode, outputs[s][1]))
     return [(s, yaml.load(o[0], yaml.CSafeLoader)) for (s, o) in outputs.iteritems()]