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

Fix mutable default argument

Browse source
This commit is contained in:
Victor Salgado 2015-09-16 16:49:45 -03:00
parent 33e79203ce
commit 54f1560d3e
3 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/ansible/inventory/dir.py
View file

@ -81,7 +81,10 @@ def get_file_parser(hostsfile, groups, loader):
class InventoryDirectory(object):
''' Host inventory parser for ansible using a directory of inventories. '''
def __init__(self, loader, groups=dict(), filename=C.DEFAULT_HOST_LIST):
def __init__(self, loader, groups=None, filename=C.DEFAULT_HOST_LIST):
if groups is None:
groups = dict()
self.names = os.listdir(filename)
self.names.sort()
self.directory = filename

5
lib/ansible/inventory/ini.py
View file

@ -38,7 +38,10 @@ class InventoryParser(object):
with their associated hosts and variable settings.
"""
def __init__(self, loader, groups=dict(), filename=C.DEFAULT_HOST_LIST):
def __init__(self, loader, groups=None, filename=C.DEFAULT_HOST_LIST):
if groups is None:
groups = dict()
self._loader = loader
self.filename = filename

4
lib/ansible/inventory/script.py
View file

@ -36,7 +36,9 @@ from ansible.module_utils.basic import json_dict_bytes_to_unicode
class InventoryScript:
''' Host inventory parser for ansible using external inventory scripts. '''
def __init__(self, loader, groups=dict(), filename=C.DEFAULT_HOST_LIST):
def __init__(self, loader, groups=None, filename=C.DEFAULT_HOST_LIST):
if groups is None:
groups = dict()
self._loader = loader
self.groups = groups