From 8a6059dce20caaceec16d4773f883aa2ddbe5ce3 Mon Sep 17 00:00:00 2001 From: Greg Price Date: Thu, 24 Jul 2008 21:42:18 -0400 Subject: [PATCH] fix another race in remctl interface This had caused some 500s with a message 'Error: Domain <...> does not exist.' svn path=/trunk/packages/sipb-xen-remote-server/; revision=721 --- files/usr/sbin/sipb-xen-remote-control | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/files/usr/sbin/sipb-xen-remote-control b/files/usr/sbin/sipb-xen-remote-control index 1ad8721..e61cb72 100755 --- a/files/usr/sbin/sipb-xen-remote-control +++ b/files/usr/sbin/sipb-xen-remote-control @@ -23,10 +23,18 @@ def main(argv): if machine_name not in vms: print >>sys.stderr, "machine '%s' is not on" % machine_name - return 2 + return 1 host = vms[machine_name]['host'] - return call(['remctl', host, 'remote', 'control'] + argv[1:]) + p = Popen(['remctl', host, 'remote', 'control'] + argv[1:], + stdout=PIPE, stderr=PIPE) + (out, err) = p.communicate() + if p.returncode == 1: + print >>sys.stderr, "machine '%s' is not on" % machine_name + return 1 + sys.stderr.write(err) + sys.stdout.write(out) + return p.returncode if __name__ == '__main__': sys.exit(main(sys.argv)) -- 1.7.9.5