Fix 'ansible-config dump --only-changed -t all' verbosity (#77898)

* Fix 'ansible-config dump --only-changed -t all' to only display headers if plugin options are changed

* changelog

* add a test
pull/77924/head
Sloane Hertel 3 years ago committed by GitHub
parent 43d650f924
commit 1214b63f4f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -0,0 +1,2 @@
bugfixes:
- ansible-config dump - Only display plugin type headers when plugin options are changed if --only-changed is specified.

@ -392,7 +392,7 @@ class ConfigCLI(CLI):
entries = [] entries = []
for setting in sorted(config): for setting in sorted(config):
changed = (config[setting].origin != 'default') changed = (config[setting].origin not in ('default', 'REQUIRED'))
if context.CLIARGS['format'] == 'display': if context.CLIARGS['format'] == 'display':
if isinstance(config[setting], Setting): if isinstance(config[setting], Setting):
@ -512,6 +512,7 @@ class ConfigCLI(CLI):
for ptype in C.CONFIGURABLE_PLUGINS: for ptype in C.CONFIGURABLE_PLUGINS:
plugin_list = self._get_plugin_configs(ptype, context.CLIARGS['args']) plugin_list = self._get_plugin_configs(ptype, context.CLIARGS['args'])
if context.CLIARGS['format'] == 'display': if context.CLIARGS['format'] == 'display':
if not context.CLIARGS['only_changed'] or plugin_list:
output.append('\n%s:\n%s' % (ptype.upper(), '=' * len(ptype))) output.append('\n%s:\n%s' % (ptype.upper(), '=' * len(ptype)))
output.extend(plugin_list) output.extend(plugin_list)
else: else:

@ -19,6 +19,10 @@ ANSIBLE_CONFIG=nonexistent.cfg ansible-config dump --only-changed -v | grep 'No
# https://github.com/ansible/ansible/pull/73715 # https://github.com/ansible/ansible/pull/73715
ANSIBLE_CONFIG=inline_comment_ansible.cfg ansible-config dump --only-changed | grep "'ansibull'" ANSIBLE_CONFIG=inline_comment_ansible.cfg ansible-config dump --only-changed | grep "'ansibull'"
# test type headers are only displayed with --only-changed -t all for changed options
env -i PATH="$PATH" PYTHONPATH="$PYTHONPATH" ansible-config dump --only-changed -t all | grep -v "CONNECTION"
env -i PATH="$PATH" PYTHONPATH="$PYTHONPATH" ANSIBLE_SSH_PIPELINING=True ansible-config dump --only-changed -t all | grep "CONNECTION"
# test the config option validation # test the config option validation
ansible-playbook validation.yml "$@" ansible-playbook validation.yml "$@"

Loading…
Cancel
Save