Commit 70d024ff by Brian Coca

show errors when missing vars file or when there is syntax issue in the file

fixes #11945
parent 464a436f
...@@ -190,7 +190,7 @@ class VariableManager: ...@@ -190,7 +190,7 @@ class VariableManager:
if play: if play:
all_vars = self._combine_vars(all_vars, play.get_vars()) all_vars = self._combine_vars(all_vars, play.get_vars())
for vars_file_item in play.get_vars_files(): for vars_file_item in play.get_vars_files():
try: try:
# create a set of temporary vars here, which incorporate the # create a set of temporary vars here, which incorporate the
...@@ -215,11 +215,6 @@ class VariableManager: ...@@ -215,11 +215,6 @@ class VariableManager:
break break
else: else:
raise AnsibleError("vars file %s was not found" % vars_file_item) raise AnsibleError("vars file %s was not found" % vars_file_item)
except AnsibleError, e:
# FIXME: get_vars should probably be taking a flag to determine
# whether or not vars files errors should be fatal at this
# stage, or just base it on whether a host was specified?
pass
except UndefinedError, e: except UndefinedError, e:
continue continue
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment