Skip to content

Commit 4d6677e

Browse files
authored
Merge pull request #77 from myii/bug/fix-map-jinja-for-salt-ssh
fix(map): avoid `defaults.merge` bug on `salt-ssh`
2 parents 1a64c23 + 6c04031 commit 4d6677e

File tree

1 file changed

+14
-16
lines changed

1 file changed

+14
-16
lines changed

redis/map.jinja

Lines changed: 14 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,19 @@
11
# -*- coding: utf-8 -*-
22
# vim: ft=jinja
33

4-
{% import_yaml 'redis/defaults.yaml' as defaults %}
5-
{% import_yaml 'redis/osfamilymap.yaml' as osfamilymap %}
6-
{% import_yaml 'redis/osfingermap.yaml' as osfingermap %}
4+
{#- Start imports as #}
5+
{%- import_yaml 'redis/defaults.yaml' as default_settings %}
6+
{%- import_yaml 'redis/osfamilymap.yaml' as osfamilymap %}
7+
{%- import_yaml 'redis/osfingermap.yaml' as osfingermap %}
78

8-
{# merge the osfamilymap #}
9-
{% set osfamily = salt['grains.filter_by'](osfamilymap, grain='os_family') or{} %}
10-
{% do salt['defaults.merge'](defaults['redis'], osfamily) %}
9+
{%- set defaults = salt['grains.filter_by'](default_settings,
10+
default='redis',
11+
merge=salt['grains.filter_by'](osfamilymap, grain='os_family',
12+
merge=salt['grains.filter_by'](osfingermap, grain='osfinger',
13+
merge=salt['pillar.get']('redis:lookup', default={})
14+
)
15+
)
16+
) %}
1117

12-
{# merge the osfingermap #}
13-
{% set osfinger = salt['grains.filter_by'](osfingermap, grain='osfinger') or {} %}
14-
{% do salt['defaults.merge'](defaults['redis'], osfinger) %}
15-
16-
{# merge the lookup #}
17-
{% set lookup = salt['pillar.get']('redis:lookup', default={}, merge=True) %}
18-
{% do salt['defaults.merge'](defaults['redis'], lookup) %}
19-
20-
{# merge all #}
21-
{% set redis_settings = salt['pillar.get']('redis', default=defaults['redis'], merge=True) %}
18+
{#- Merge the redis pillar #}
19+
{%- set redis = salt['pillar.get']('redis', default=defaults, merge=True) %}

0 commit comments

Comments
 (0)