Changeset 2513 for trunk/packages/invirt-remote/python/remote

Timestamp:

Oct 24, 2009, 8:40:08 PM (15 years ago)

Author:

adehnert

Message:

Minor cleanup of monocast and LVM load balancer

File:

• trunk/packages/invirt-remote/python/remote/monocast.py (modified) (3 diffs)

trunk/packages/invirt-remote/python/remote/monocast.py

@@ -5 +5 @@
 hostnames = [ h.hostname for h in config.hosts ]
 
-def monocast(args, hosts = hostnames, randomize = True):
+def monocast(args, hostnames = hostnames, randomize = True):
     """
     Given a command and a list of hostnames or IPs, issue the command to each
@@ -18 +18 @@
     """
     if(randomize):
-        hosts = random.sample(hosts, len(hosts))
+        hostnames = random.sample(hostnames, len(hostnames))
     hostsdown = []
-    for host in hosts:
+    for host in hostnames:
         pipe = Popen(['remctl', host, 'remote', 'web'] + args, stdout=PIPE, stderr=PIPE)
         output = pipe.communicate()
@@ -27 +27 @@
                 hostsdown.append(host)
             else:
-                #raise RuntimeError("remctl to host %s returned non-zero exit status %d; stderr:\n%s"
-                #                   % (host, pipe.returncode, output[1]))
                 return (host, hostsdown, pipe.returncode,) + output
         else: