If we had any errors in setup in one of the hosts, exit with non-zero
authorRené Nussbaumer <rn@google.com>
Thu, 9 Sep 2010 11:13:08 +0000 (13:13 +0200)
committerRené Nussbaumer <rn@google.com>
Thu, 9 Sep 2010 12:43:22 +0000 (14:43 +0200)
Signed-off-by: René Nussbaumer <rn@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com>

tools/setup-ssh

index e5a512b..c6c0bd6 100755 (executable)
@@ -304,6 +304,8 @@ def main():
 
   SetupLogging(options)
 
+  errs = 0
+
   all_keys = LoadPrivateKeys(options)
 
   passwd = None
@@ -345,14 +347,18 @@ def main():
         SetupSSH(transport)
         SetupNodeDaemon(transport)
       except errors.GenericError, err:
-        logging.error("While doing setup on host %s an error occured: %s",
+        logging.error("While doing setup on host %s an error occurred: %s",
                       host, err)
+        errs += 1
     finally:
       transport.close()
       # this is needed for compatibility with older Paramiko or Python
       # versions
       transport.join()
 
+    if errs > 0:
+      sys.exit(1)
+
 
 if __name__ == "__main__":
   main()