cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Undeprecated the skip argument for first_found. Fixes #58942. Fixes #59949. (#60161)

Browse source
This commit is contained in:
Matt Martz 2019-08-08 13:55:11 -05:00 committed by GitHub
parent c954c07271
commit 707e33793d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
changelogs/fragments/59949-undeprecate-first-found-skip.yml Normal file
View file

@ -0,0 +1,4 @@
bugfixes:
- first_found - Un-deprecate ``skip``, as the alternative of ``errors`` does
not work with ``with_first_found`` and only use of ``lookup``
(https://github.com/ansible/ansible/issues/58942)

7
lib/ansible/plugins/lookup/first_found.py
View file

@ -29,10 +29,6 @@ DOCUMENTATION = """
type: boolean type: boolean
default: False default: False
description: Return an empty list if no file is found, instead of an error. description: Return an empty list if no file is found, instead of an error.
deprecated:
why: A generic that applies to all errors exists for all lookups.
version: "2.8"
alternative: The generic ``errors=ignore``
""" """
EXAMPLES = """ EXAMPLES = """
@ -130,9 +126,6 @@ class LookupModule(LookupBase):
for term in terms: for term in terms:
if isinstance(term, dict): if isinstance(term, dict):
if 'skip' in term:
self._display.deprecated('Use errors="ignore" instead of skip', version='2.12')
files = term.get('files', []) files = term.get('files', [])
paths = term.get('paths', []) paths = term.get('paths', [])
skip = boolean(term.get('skip', False), strict=False) skip = boolean(term.get('skip', False), strict=False)