wbrawner/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

set _hosts on access if None (#31111)

Browse source 
set _hosts on access if None to bpyass srlz10n issues to fix #30903

(cherry picked from commit cf3414d7d7)
This commit is contained in:
Brian Coca 2017-10-03 11:59:05 -04:00 committed by Matt Clay
parent b81270f7d5
commit f09c1211a7
1 changed files with 11 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
lib/ansible/inventory/group.py
View file

@ -18,7 +18,6 @@ from __future__ import (absolute_import, division, print_function)
__metaclass__ = type
from ansible.errors import AnsibleError
from ansible.utils.vars import combine_vars
class Group:
@ -31,7 +30,7 @@ class Group:
self.depth = 0
self.name = name
self.hosts = []
self._hosts = set()
self._hosts = None
self.vars = {}
self.child_groups = []
self.parent_groups = []
@ -72,9 +71,8 @@ class Group:
self.name = data.get('name')
self.vars = data.get('vars', dict())
self.depth = data.get('depth', 0)
self.hosts = data.get('hosts', {})
self._hosts = set(self.hosts)
self.hosts = data.get('hosts', [])
self._hosts = None
parent_groups = data.get('parent_groups', [])
for parent_data in parent_groups:
@ -82,6 +80,12 @@ class Group:
g.deserialize(parent_data)
self.parent_groups.append(g)
@property
def host_names(self):
if self._hosts is None:
self._hosts = set(self.hosts)
return self._hosts
def get_name(self):
return self.name
@ -119,7 +123,7 @@ class Group:
raise AnsibleError("The group named '%s' has a recursive dependency loop." % self.name)
def add_host(self, host):
if host.name not in self._hosts:
if host.name not in self.host_names:
self.hosts.append(host)
self._hosts.add(host.name)
host.add_group(self)
@ -127,7 +131,7 @@ class Group:
def remove_host(self, host):
if host.name in self._hosts:
if host.name in self.host_names:
self.hosts.remove(host)
self._hosts.remove(host.name)
host.remove_group(self)