diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2017-09-22 09:23:52 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-22 09:23:52 -0700 |
commit | 9240e0d006198f910ea01d34d57f04f9ecd8e7e8 (patch) | |
tree | f95f8a2fb02155f7978467088b06a3580e25b0b9 /roles/openshift_sanitize_inventory/tasks/deprecations.yml | |
parent | fe2f31cb1ea499d2683e7e30679eef8ac69857ad (diff) | |
parent | b2fdc1cb5e38e39250af702d1e19ed691e4df7d5 (diff) | |
download | openshift-9240e0d006198f910ea01d34d57f04f9ecd8e7e8.tar.gz openshift-9240e0d006198f910ea01d34d57f04f9ecd8e7e8.tar.bz2 openshift-9240e0d006198f910ea01d34d57f04f9ecd8e7e8.tar.xz openshift-9240e0d006198f910ea01d34d57f04f9ecd8e7e8.zip |
Merge pull request #5176 from ewolinetz/deprecate_log_met_vars
Automatic merge from submit-queue
Creating structure to warn for use of deprecated variables and set th…
…em in a single location before they are no longer honored
Implementation of https://trello.com/c/XKOGHSkP/460-3-deprecate-openshifthostedloggingmetrics-installer-variables
Tried to implement this in a way such that we can easily and cleanly add/remove variables in the future for any role we choose.
Addresses:
https://bugzilla.redhat.com/show_bug.cgi?id=1449812
https://bugzilla.redhat.com/show_bug.cgi?id=1464349
Diffstat (limited to 'roles/openshift_sanitize_inventory/tasks/deprecations.yml')
-rw-r--r-- | roles/openshift_sanitize_inventory/tasks/deprecations.yml | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/roles/openshift_sanitize_inventory/tasks/deprecations.yml b/roles/openshift_sanitize_inventory/tasks/deprecations.yml new file mode 100644 index 000000000..94d3acffc --- /dev/null +++ b/roles/openshift_sanitize_inventory/tasks/deprecations.yml @@ -0,0 +1,21 @@ +--- + +- name: Check for usage of deprecated variables + set_fact: + __deprecation_message: "{{ __deprecation_message | default([]) }} + ['{{ __deprecation_header }} {{ item }} is a deprecated variable and will be no longer be used in the next minor release. Please update your inventory accordingly.']" + when: + - hostvars[inventory_hostname][item] is defined + with_items: "{{ __warn_deprecated_vars }}" + +- block: + - debug: msg="{{__deprecation_message}}" + - pause: + seconds: "{{ 10 }}" + when: + - __deprecation_message | default ('') | length > 0 + +# for with_fileglob Ansible resolves the path relative to the roles/<rolename>/files directory +- name: Assign deprecated variables to correct counterparts + include: "{{ item }}" + with_fileglob: + - "../tasks/__deprecations_*.yml" |