[padb] r323 committed - Only check for pdsh being installed if we are actually going to use it...

padb at googlecode.com padb at googlecode.com
Thu Nov 5 22:17:00 GMT 2009


Revision: 323
Author: apittman
Date: Thu Nov  5 14:13:38 2009
Log: Only check for pdsh being installed if we are actually going to use  
it, for
single host jobs where the target processes are local it's not needed.

http://code.google.com/p/padb/source/detail?r=323

Modified:
  /trunk/src/padb

=======================================
--- /trunk/src/padb	Thu Nov  5 14:11:00 2009
+++ /trunk/src/padb	Thu Nov  5 14:13:38 2009
@@ -3039,14 +3039,15 @@
      # way can be found in the future.
      if ( defined $pcmd{host_list} and not defined $pcmd{command} ) {

-        if ( not find_exe('pdsh') ) {
-            print
-              "$conf{rmgr} resource manager requires pdsh to be  
installed\n";
-            return;
-        }
-
          my @hosts = @{ $pcmd{host_list} };
          if ( $hosts[0] ne hostname() or @hosts > 1 ) {
+
+            if ( not find_exe('pdsh') ) {
+                print
+"$conf{rmgr} resource manager on multiple or remote hosts requires pdsh to  
be installed\n";
+                return;
+            }
+
              $pcmd{require_inner_callback} = 1;
              my $hlist = join q{,}, @hosts;
              $pcmd{command} = "pdsh -w $hlist";




More information about the padb-devel mailing list