diff options
author | James Cammarata <jimi@sngx.net> | 2016-11-21 02:03:16 -0600 |
---|---|---|
committer | James Cammarata <jimi@sngx.net> | 2016-11-21 10:39:42 -0600 |
commit | 0df3767d4d6a522141d6916e5bbe4babe3e4929b (patch) | |
tree | 0ccdb118a9682c439e7bb87d81d1d36e34d1caf1 | |
parent | 69649358b10e7b30ed792906db302eaec9640c7a (diff) | |
download | ansible-0df3767d4d6a522141d6916e5bbe4babe3e4929b.tar.gz |
When iterating over hostvars yield the hostname not the host object
Also fixes HostVars to raise the correct jinja2 error type.
Fixes #16836
-rw-r--r-- | lib/ansible/vars/hostvars.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/ansible/vars/hostvars.py b/lib/ansible/vars/hostvars.py index 36adf54906..74262b2ed4 100644 --- a/lib/ansible/vars/hostvars.py +++ b/lib/ansible/vars/hostvars.py @@ -21,7 +21,7 @@ __metaclass__ = type import collections -from jinja2 import Undefined as j2undefined +from jinja2.exceptions import UndefinedError from ansible import constants as C from ansible.template import Templar @@ -73,7 +73,7 @@ class HostVars(collections.Mapping): ''' host = self._find_host(host_name) if host is None: - raise j2undefined + raise UndefinedError("%s not found in hostvars" % host_name) return self._variable_manager.get_vars(loader=self._loader, host=host, include_hostvars=False) @@ -102,7 +102,7 @@ class HostVars(collections.Mapping): def __iter__(self): for host in self._inventory.get_hosts(ignore_limits=True, ignore_restrictions=True): - yield host + yield host.name def __len__(self): return len(self._inventory.get_hosts(ignore_limits=True, ignore_restrictions=True)) |