catch syntax/logic issue correctly -> thanks to greg

[certmaster.git] / certmaster / utils.py

diff --git a/certmaster/utils.py b/certmaster/utils.py
index 6b79ce7..02c28e0 100644 (file)
--- a/certmaster/utils.py
+++ b/certmaster/utils.py
@@ -139,7 +139,7 @@ def create_minion_keys(hostname=None):
         raise codes.CMException("Could not determine a hostname other than localhost")
     else:
         # use lowercase letters for hostnames
-        hostname = hostname.lower()
+        hn = hn.lower()
 
     key_file = '%s/%s.pem' % (cert_dir, hn)
     csr_file = '%s/%s.csr' % (cert_dir, hn)
@@ -172,8 +172,8 @@ def create_minion_keys(hostname=None):
             # print "DEBUG: submitting CSR to certmaster: %s" % master_uri
             log.debug("submitting CSR: %s  to certmaster %s" % (csr_file, master_uri))
             result, cert_string, ca_cert_string = submit_csr_to_master(csr_file, master_uri)
-        except socket.gaierror, e:
-            raise codes.CMException, "Could not locate certmaster at %s" % master_uri
+        except socket.error, e:
+            log.warning("Could not locate certmaster at %s" % master_uri)
 
         # logging here would be nice
         if not result: