From 879dc3a687fde49f532869b3de3f19ebbfc977ea Mon Sep 17 00:00:00 2001 From: James Cammarata Date: Sat, 4 Jun 2016 18:53:47 -0500 Subject: [PATCH] Mark implicitly hosts as such and exclude them from the all group Fixes #16059 --- lib/ansible/inventory/__init__.py | 5 +++++ lib/ansible/inventory/group.py | 4 ++++ lib/ansible/inventory/host.py | 3 +++ 3 files changed, 12 insertions(+) diff --git a/lib/ansible/inventory/__init__.py b/lib/ansible/inventory/__init__.py index 6d93fb6dbcd..27198f1c9ed 100644 --- a/lib/ansible/inventory/__init__.py +++ b/lib/ansible/inventory/__init__.py @@ -445,10 +445,14 @@ class Inventory(object): for group in groups.values(): if pattern == 'all': for host in group.get_hosts(): + if host.implicit: + continue __append_host_to_results(host) else: if self._match(group.name, pattern) and group.name not in ('all', 'ungrouped'): for host in group.get_hosts(): + if host.implicit: + continue __append_host_to_results(host) else: matching_hosts = self._match_list(group.get_hosts(), 'name', pattern) @@ -463,6 +467,7 @@ class Inventory(object): def _create_implicit_localhost(self, pattern): new_host = Host(pattern) new_host.address = "127.0.0.1" + new_host.implicit = True new_host.vars = self.get_host_vars(new_host) new_host.set_variable("ansible_connection", "local") if "ansible_python_interpreter" not in new_host.vars: diff --git a/lib/ansible/inventory/group.py b/lib/ansible/inventory/group.py index 482fe2a18fc..63c297aaa47 100644 --- a/lib/ansible/inventory/group.py +++ b/lib/ansible/inventory/group.py @@ -140,10 +140,14 @@ class Group: for kk in kid_hosts: if kk not in seen: seen[kk] = 1 + if self.name == 'all' and kk.implicit: + continue hosts.append(kk) for mine in self.hosts: if mine not in seen: seen[mine] = 1 + if self.name == 'all' and mine.implicit: + continue hosts.append(mine) return hosts diff --git a/lib/ansible/inventory/host.py b/lib/ansible/inventory/host.py index 6263dcbc80d..36cc90f5377 100644 --- a/lib/ansible/inventory/host.py +++ b/lib/ansible/inventory/host.py @@ -60,6 +60,7 @@ class Host: uuid=self._uuid, gathered_facts=self._gathered_facts, groups=groups, + implicit=self.implicit, ) def deserialize(self, data): @@ -69,6 +70,7 @@ class Host: self.vars = data.get('vars', dict()) self.address = data.get('address', '') self._uuid = data.get('uuid', uuid.uuid4()) + self.implicit= data.get('implicit', False) groups = data.get('groups', []) for group_data in groups: @@ -89,6 +91,7 @@ class Host: self._gathered_facts = False self._uuid = uuid.uuid4() + self.implicit = False def __repr__(self): return self.get_name()