internet/community.general
3
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2026-04-05 10:13:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Test for play.vars_files == None, avoiding nasty traceback. Ignores for now. Could warn.

Browse Source 
Playbook test for no vars_files with len(), but that excepts if play.vars_files==None, as can happen when there's a vars_files section with no vars files listed. What is the ansible way: ignore, warn, or fail with message (instead of traceback)?
This commit is contained in:
Brad Olson
2012-05-29 10:09:38 -04:00
parent c03fec4864
commit 32b9ea689a
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/ansible/playbook/__init__.py
View File

@@ -280,8 +280,9 @@ class PlayBook(object):
rc = self._do_setup_step(play) # pattern, vars, user, port, sudo, sudo_user, transport, None)
# now with that data, handle contentional variable file imports!
if len(play.vars_files) > 0:
if play.vars_files and len(play.vars_files) > 0:
rc = self._do_setup_step(play, play.vars_files)
#else: warn "You have a vars_files section but didn't state any vars files??
# run all the top level tasks, these get run on every node
for task in play.tasks():