Skip to content

Commit

Permalink
Properly set the default value for pillar_merge_lists
Browse files Browse the repository at this point in the history
  • Loading branch information
anlutro authored and basepi committed Feb 6, 2016
1 parent c7472ff commit 2243f25
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions salt/pillar/__init__.py
Expand Up @@ -535,7 +535,7 @@ def render_pstate(self, sls, saltenv, mods, defaults=None):
nstate,
self.merge_strategy,
self.opts.get('renderer', 'yaml'),
self.opts.get('pillar_merge_lists', 'False'))
self.opts.get('pillar_merge_lists', False))

if err:
errors += err
Expand Down Expand Up @@ -573,7 +573,7 @@ def render_pillar(self, matches):
pstate,
self.merge_strategy,
self.opts.get('renderer', 'yaml'),
self.opts.get('pillar_merge_lists', 'False'))
self.opts.get('pillar_merge_lists', False))

return pillar, errors

Expand Down Expand Up @@ -660,7 +660,7 @@ def ext_pillar(self, pillar, pillar_dirs):
ext,
self.merge_strategy,
self.opts.get('renderer', 'yaml'),
self.opts.get('pillar_merge_lists', 'False'))
self.opts.get('pillar_merge_lists', False))
ext = None
return pillar

Expand All @@ -678,7 +678,7 @@ def compile_pillar(self, ext=True, pillar_dirs=None):
self.opts['pillar'],
self.merge_strategy,
self.opts.get('renderer', 'yaml'),
self.opts.get('pillar_merge_lists', 'False'))
self.opts.get('pillar_merge_lists', False))
else:
matches = self.top_matches(top)
pillar, errors = self.render_pillar(matches)
Expand Down
2 changes: 1 addition & 1 deletion salt/pillar/git_pillar.py
Expand Up @@ -262,7 +262,7 @@ def ext_pillar(minion_id, repo, pillar_dirs):
)
merge_lists = __opts__.get(
'pillar_merge_lists',
'False'
False
)
for pillar_dir, env in six.iteritems(pillar.pillar_dirs):
log.debug(
Expand Down

0 comments on commit 2243f25

Please sign in to comment.