fix vars hostname fallback

also made it optimistic, rely on exceptions instead of copmlex if chains

(cherry picked from commit 11dbed1350)
pull/42228/head
Brian Coca 6 years ago committed by Kevin Breit
parent 5cc2efb69b
commit e9cbb19b33

@ -0,0 +1,2 @@
bugfixes:
- correctly check hostvars for vars term https://github.com/ansible/ansible/pull/41819

@ -78,15 +78,15 @@ class LookupModule(LookupBase):
raise AnsibleError('Invalid setting identifier, "%s" is not a string, its a %s' % (term, type(term))) raise AnsibleError('Invalid setting identifier, "%s" is not a string, its a %s' % (term, type(term)))
try: try:
if term in myvars: try:
value = myvars[term] value = myvars[term]
elif 'hostvars' in myvars and term in myvars['hostvars']: except KeyError:
# maybe it is a host var? try:
value = myvars['hostvars'][term] value = myvars['hostvars'][myvars['inventory_hostname']][term]
else: except KeyError:
raise AnsibleUndefinedVariable('No variable found with this name: %s' % term) raise AnsibleUndefinedVariable('No variable found with this name: %s' % term)
ret.append(self._templar.template(value, fail_on_undefined=True))
ret.append(self._templar.template(value, fail_on_undefined=True))
except AnsibleUndefinedVariable: except AnsibleUndefinedVariable:
if default is not None: if default is not None:
ret.append(default) ret.append(default)

Loading…
Cancel
Save