diff options
| author | Michael Gugino <mgugino@redhat.com> | 2017-10-03 13:03:19 -0400 |
|---|---|---|
| committer | Michael Gugino <mgugino@redhat.com> | 2017-10-03 13:03:19 -0400 |
| commit | 4b0f5219f4ec3eeb5433fc9f985b1e067b581276 (patch) | |
| tree | 8f3d442273d7b2d5313a810cc7d44d31fe4e6ca3 /filter_plugins | |
| parent | ee0a668a8848b6d707a57de1b908584cf0b28d5d (diff) | |
| download | openshift-4b0f5219f4ec3eeb5433fc9f985b1e067b581276.tar.gz openshift-4b0f5219f4ec3eeb5433fc9f985b1e067b581276.tar.bz2 openshift-4b0f5219f4ec3eeb5433fc9f985b1e067b581276.tar.xz openshift-4b0f5219f4ec3eeb5433fc9f985b1e067b581276.zip | |
Fix hosted selector variable migration
Currently, if undefined, openshift_hosted_router_selector and
openshift_hosted_registry_selector as set to the value
of openshift_hosted_infra_selector.
The set_fact logic overrides variable migration inside of
oo_filters meant to utilize the deprecated variables
openshift_router_selector and openshift_registiry_selector,
respectively.
This commit corrects the set_fact logic and removes the
unused migration from oo_filters.
Fixes: https://github.com/openshift/openshift-ansible/issues/5615
Diffstat (limited to 'filter_plugins')
| -rw-r--r-- | filter_plugins/oo_filters.py | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/filter_plugins/oo_filters.py b/filter_plugins/oo_filters.py index f0f250480..83a05370a 100644 --- a/filter_plugins/oo_filters.py +++ b/filter_plugins/oo_filters.py @@ -707,11 +707,6 @@ def oo_openshift_env(hostvars): if regex.match(key): facts[key] = hostvars[key] - migrations = {'openshift_router_selector': 'openshift_hosted_router_selector', - 'openshift_registry_selector': 'openshift_hosted_registry_selector'} - for old_fact, new_fact in migrations.items(): - if old_fact in facts and new_fact not in facts: - facts[new_fact] = facts[old_fact] return facts |