summaryrefslogtreecommitdiffstats
path: root/roles/openshift_facts/library
diff options
context:
space:
mode:
authorJason DeTiberus <jdetiber@redhat.com>2017-02-27 15:25:03 -0500
committerScott Dodson <sdodson@redhat.com>2017-06-01 15:25:37 -0400
commitc6837b8b6b09f927a3ea9b40f3c10bdcf094d9ff (patch)
tree000f6f50fd53e7f9573dc7ccd3139c50c129826a /roles/openshift_facts/library
parent7aaeae708369300d770d8282a9cd74a4db775aa0 (diff)
downloadopenshift-c6837b8b6b09f927a3ea9b40f3c10bdcf094d9ff.tar.gz
openshift-c6837b8b6b09f927a3ea9b40f3c10bdcf094d9ff.tar.bz2
openshift-c6837b8b6b09f927a3ea9b40f3c10bdcf094d9ff.tar.xz
openshift-c6837b8b6b09f927a3ea9b40f3c10bdcf094d9ff.zip
Do not attempt to override openstack nodename
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1392519
Diffstat (limited to 'roles/openshift_facts/library')
-rwxr-xr-xroles/openshift_facts/library/openshift_facts.py12
1 files changed, 7 insertions, 5 deletions
diff --git a/roles/openshift_facts/library/openshift_facts.py b/roles/openshift_facts/library/openshift_facts.py
index 514c06500..cfe092a28 100755
--- a/roles/openshift_facts/library/openshift_facts.py
+++ b/roles/openshift_facts/library/openshift_facts.py
@@ -193,8 +193,7 @@ def hostname_valid(hostname):
"""
if (not hostname or
hostname.startswith('localhost') or
- hostname.endswith('localdomain') or
- hostname.endswith('novalocal')):
+ hostname.endswith('localdomain')):
return False
return True
@@ -1041,10 +1040,13 @@ def set_sdn_facts_if_unset(facts, system_facts):
def set_nodename(facts):
""" set nodename """
if 'node' in facts and 'common' in facts:
- if 'cloudprovider' in facts and facts['cloudprovider']['kind'] == 'openstack':
- facts['node']['nodename'] = facts['provider']['metadata']['hostname'].replace('.novalocal', '')
- elif 'cloudprovider' in facts and facts['cloudprovider']['kind'] == 'gce':
+ if 'cloudprovider' in facts and facts['cloudprovider']['kind'] == 'gce':
facts['node']['nodename'] = facts['provider']['metadata']['instance']['hostname'].split('.')[0]
+
+ # TODO: The openstack cloudprovider nodename setting was too opinionaed.
+ # It needs to be generalized before it can be enabled again.
+ # elif 'cloudprovider' in facts and facts['cloudprovider']['kind'] == 'openstack':
+ # facts['node']['nodename'] = facts['provider']['metadata']['hostname'].replace('.novalocal', '')
else:
facts['node']['nodename'] = facts['common']['hostname'].lower()
return facts