diff options
author | Scott Dodson <sdodson@redhat.com> | 2017-02-13 15:57:15 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-02-13 15:57:15 -0500 |
commit | acea04a67bd3aa62001af4fe426a894856609d2a (patch) | |
tree | 56b82be0eb6a801609f4a8545a23fd43409cef1c /roles/openshift_facts/library/openshift_facts.py | |
parent | 671062646c9a409470be374408f104352af825f0 (diff) | |
parent | cba1d9e8bcc087c550ce3a4221cd08dc6eac1240 (diff) | |
download | openshift-acea04a67bd3aa62001af4fe426a894856609d2a.tar.gz openshift-acea04a67bd3aa62001af4fe426a894856609d2a.tar.bz2 openshift-acea04a67bd3aa62001af4fe426a894856609d2a.tar.xz openshift-acea04a67bd3aa62001af4fe426a894856609d2a.zip |
Merge pull request #3343 from sdodson/bz1419654
BZ1419654 Remove legacy config_base fallback to /etc/openshift
Diffstat (limited to 'roles/openshift_facts/library/openshift_facts.py')
-rwxr-xr-x | roles/openshift_facts/library/openshift_facts.py | 24 |
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', |