Add hostname/nodename length check

OpenShift will not allow nodes to register if the nodename is > 63 chars.
This commit verifies that nodename is not set to a potential hostname
that is greater than 63 characters.  This commit also updates logic
to check `openshift_hostname` to check for > 63 chars.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1467790

1 files changed, 3 insertions, 1 deletions

diff --git a/roles/openshift_facts/library/openshift_facts.py b/roles/openshift_facts/library/openshift_facts.py
index 844d77255..251d1dfb4 100755
--- a/roles/openshift_facts/library/openshift_facts.py
+++ b/roles/openshift_facts/library/openshift_facts.py
@@ -193,7 +193,9 @@ def hostname_valid(hostname):
     """
     if (not hostname or
             hostname.startswith('localhost') or
-            hostname.endswith('localdomain')):
+            hostname.endswith('localdomain') or
+            # OpenShift will not allow a node with more than 63 chars in name.
+            len(hostname) > 63):
         return False
 
     return True