Optimizes using fnmatch only if needed

Skips fnmatch'ing if there is only a '*' in filter.
reviewable/pr18780/r1
Michel Blanc 12 years ago
parent d80e78a9ae
commit 981add2561

@ -1140,7 +1140,7 @@ def run_setup(module):
setup_result = { 'ansible_facts': {} } setup_result = { 'ansible_facts': {} }
for (k,v) in setup_options.items(): for (k,v) in setup_options.items():
if fnmatch.fnmatch(k, module.params['filter']): if module.params['filter'] == '*' or fnmatch.fnmatch(k, module.params['filter']):
setup_result['ansible_facts'][k] = v setup_result['ansible_facts'][k] = v
# hack to keep --verbose from showing all the setup module results # hack to keep --verbose from showing all the setup module results

Loading…
Cancel
Save