summaryrefslogtreecommitdiffstats
path: root/roles
diff options
context:
space:
mode:
authorScott Dodson <sdodson@redhat.com>2017-02-13 13:55:27 -0500
committerScott Dodson <sdodson@redhat.com>2017-02-13 15:33:01 -0500
commitcba1d9e8bcc087c550ce3a4221cd08dc6eac1240 (patch)
tree64a5f2b25483017e1979cff57e4ec4fbc5fc3a44 /roles
parentd9a6627ac9e019f699b749f427e107514f96d9b0 (diff)
downloadopenshift-cba1d9e8bcc087c550ce3a4221cd08dc6eac1240.tar.gz
openshift-cba1d9e8bcc087c550ce3a4221cd08dc6eac1240.tar.bz2
openshift-cba1d9e8bcc087c550ce3a4221cd08dc6eac1240.tar.xz
openshift-cba1d9e8bcc087c550ce3a4221cd08dc6eac1240.zip
Fix Bug 1419654 Remove legacy config_base fallback to /etc/openshift
If a host had /etc/openshift but not /etc/origin we were setting the config_base to /etc/openshift in some places but not all. This code was transitional in order to migrate between 3.0 and 3.1. Given that current playbooks are only supported when moving from the previous version to current version this should no longer be necessary.
Diffstat (limited to 'roles')
-rwxr-xr-xroles/openshift_facts/library/openshift_facts.py24
1 files changed, 5 insertions, 19 deletions
diff --git a/roles/openshift_facts/library/openshift_facts.py b/roles/openshift_facts/library/openshift_facts.py
index 97b3edd6e..ef7f159c5 100755
--- a/roles/openshift_facts/library/openshift_facts.py
+++ b/roles/openshift_facts/library/openshift_facts.py
@@ -788,8 +788,8 @@ def set_etcd_facts_if_unset(facts):
def set_deployment_facts_if_unset(facts):
""" Set Facts that vary based on deployment_type. This currently
- includes common.service_type, common.config_base, master.registry_url,
- node.registry_url, node.storage_plugin_deps
+ includes common.service_type, master.registry_url, node.registry_url,
+ node.storage_plugin_deps
Args:
facts (dict): existing facts
@@ -809,22 +809,6 @@ def set_deployment_facts_if_unset(facts):
elif deployment_type in ['enterprise']:
service_type = 'openshift'
facts['common']['service_type'] = service_type
- if 'config_base' not in facts['common']:
- config_base = '/etc/origin'
- if deployment_type in ['enterprise']:
- config_base = '/etc/openshift'
- # Handle upgrade scenarios when symlinks don't yet exist:
- if not os.path.exists(config_base) and os.path.exists('/etc/openshift'):
- config_base = '/etc/openshift'
- facts['common']['config_base'] = config_base
- if 'data_dir' not in facts['common']:
- data_dir = '/var/lib/origin'
- if deployment_type in ['enterprise']:
- data_dir = '/var/lib/openshift'
- # Handle upgrade scenarios when symlinks don't yet exist:
- if not os.path.exists(data_dir) and os.path.exists('/var/lib/openshift'):
- data_dir = '/var/lib/openshift'
- facts['common']['data_dir'] = data_dir
if 'docker' in facts:
deployment_type = facts['common']['deployment_type']
@@ -2001,7 +1985,9 @@ class OpenShiftFacts(object):
client_binary='oc', admin_binary='oadm',
dns_domain='cluster.local',
install_examples=True,
- debug_level=2)
+ debug_level=2,
+ config_base='/etc/origin',
+ data_dir='/var/lib/origin')
if 'master' in roles:
defaults['master'] = dict(api_use_ssl=True, api_port='8443',