Merge pull request #3 from ryanwalder/fix-debian-directory-bug

Add `zones_directory` variable to map

Author: joe-bowman

bind/config.sls

@@ -45,6 +45,17 @@ named_directory:
     - require:
       - pkg: bind
 
+bind_zones_directory:
+  file.directory:
+    - name: {{ map.named_directory }}/{{ map.zones_directory }}
+    - user: {{ salt['pillar.get']('bind:config:user', map.user) }}
+    - group: {{ salt['pillar.get']('bind:config:group', map.group) }}
+    - mode: 775
+    - makedirs: True
+    - require:
+      - pkg: bind
+      - file: named_directory
+
 bind_config:
   file.managed:
     - name: {{ map.config }}
@@ -120,6 +131,7 @@ bind_options_config:
     - context:
         key_directory: {{ map.key_directory }}
         named_directory: {{ map.named_directory }}
+        zones_directory: {{ map.zones_directory }}
     - require:
       - pkg: bind
     - watch_in:
@@ -197,7 +209,7 @@ bind_rndc_client_config:
 {% if file and zone_data['type'] == 'master' -%}
 zones{{ dash_view }}-{{ zone }}{{ '.include' if serial_auto else ''}}:
   file.managed:
-    - name: {{ map.named_directory }}/{{ file }}{{ '.include' if serial_auto else ''}}
+    - name: {{ map.named_directory }}/{{ map.zones_directory }}/{{ file }}{{ '.include' if serial_auto else ''}}
     - source: {{ zone_source }}
     - template: jinja
     {% if zone_records != {} %}
@@ -214,6 +226,7 @@ zones{{ dash_view }}-{{ zone }}{{ '.include' if serial_auto else ''}}:
       - service: bind
     - require:
       - file: named_directory
+      - file: bind_zones_directory
 
 {% if serial_auto %}
 zones{{ dash_view }}-{{ zone }}:
@@ -222,9 +235,9 @@ zones{{ dash_view }}-{{ zone }}:
     - update: True
     - zone: zones{{ dash_view }}-{{ zone }}
     - watch:
-      - file: {{ map.named_directory }}/{{ file }}.include
+      - file: {{ map.named_directory }}/{{ map.zones_directory }}/{{ file }}.include
   file.managed:
-    - name: {{ map.named_directory }}/{{ file }}
+    - name: {{ map.named_directory }}/{{ map.zones_directory }}/{{ file }}
     - require:
       - module: zones{{ dash_view }}-{{ zone }}
     - source: {{ zone_source }}
@@ -233,7 +246,7 @@ zones{{ dash_view }}-{{ zone }}:
     - context:
       zone: zones{{ dash_view }}-{{ zone }}
       soa: {{ salt['pillar.get']("bind:available_zones:" + zone + ":soa") }}
-      include: {{ file }}.include
+      include: {{ map.named_directory }}/{{ map.zones_directory }}/{{ file }}.include
     {% endif %}
     - user: {{ salt['pillar.get']('bind:config:user', map.user) }}
     - group: {{ salt['pillar.get']('bind:config:group', map.group) }}
@@ -242,11 +255,12 @@ zones{{ dash_view }}-{{ zone }}:
       - service: bind
     - require:
       - file: named_directory
+      - file: bind_zones_directory
 {% endif %}
 {% if zone_data['dnssec'] is defined and zone_data['dnssec'] -%}
 signed{{ dash_view }}-{{ zone }}:
   cmd.run:
-    - cwd: {{ map.named_directory }}
+    - cwd: {{ map.named_directory }}/{{ map.zones_directory }}
     - name: zonesigner -zone {{ zone }} {{ file }}
     - prereq:
       - file: zones{{ dash_view }}-{{ zone }}

bind/files/named.conf.local.jinja

@@ -21,9 +21,9 @@ zone "{{ key }}" {
   };
   {% else -%}
   {% if args['dnssec'] is defined and args['dnssec'] -%}
-  file "{{ map.named_directory }}/{{ file }}.signed";
+  file "{{ map.named_directory }}/{{ map.zones_directory }}/{{ file }}.signed";
   {% else -%}
-  file "{{ map.named_directory }}/{{ file }}";
+  file "{{ map.named_directory }}/{{ map.zones_directory }}/{{ file }}";
   {%- endif %}
   {% if args['auto-dnssec'] is defined -%}
   auto-dnssec {{ args['auto-dnssec'] }};

bind/map.jinja

@@ -12,7 +12,8 @@
         'default_zones_config': '/etc/bind/named.conf.default-zones',
         'logging_config': '/etc/bind/named.conf.logging',
         'rndc_client_config': '/etc/bind/rndc.conf',
-        'named_directory': '/var/cache/bind/zones',
+        'named_directory': '/var/cache/bind',
+        'zones_directory': 'zones',
         'chroot_dir': '',
         'log_dir': '/var/log/bind9',
         'log_mode': '644',
@@ -36,6 +37,7 @@
         'local_config': '/etc/named.conf.local',
         'default_config': '/etc/sysconfig/named',
         'named_directory': '/var/named',
+        'zones_directory': 'zones',
         'chroot_dir': '',
         'log_dir': '/var/log/named',
         'log_mode': '640',
@@ -63,6 +65,7 @@
         'config': '/etc/named.conf',
         'local_config': '/etc/named.conf.local',
         'named_directory': '/var/named',
+        'zones_directory': 'zones',
         'chroot_dir': '',
         'log_dir': '/var/log/named',
         'log_mode': '640',
@@ -82,6 +85,7 @@
         'config': '/usr/local/etc/namedb/named.conf',
         'local_config': '/usr/local/etc/namedb/named.conf.local',
         'named_directory': '/usr/local/etc/namedb/working',
+        'zones_directory': 'zones',
         'chroot_dir': '',
         'log_dir': '/var/log/named',
         'log_mode': '660',
@@ -102,6 +106,7 @@
         'local_config': '/etc/named.d/named.conf.local',
         'default_config': '/etc/sysconfig/named',
         'named_directory': '/var/lib/named',
+        'zones_directory': 'zones',
         'chroot_dir': '/var/lib/named',
         'log_dir': '/var/log',
         'log_mode': '660',