Merge pull request #61 from 0xf10e/master

iteritems() -> items(), fixes saltstack-formulas/bind-formula#30

README.rst

@@ -24,6 +24,10 @@ Install the bind package and start the bind service.
 ---------------
 
 Manage the bind configuration file.
+Zone files are not generated by this state
+rather than taken from `salt://zones`.
+See `pillar.example` for how to overwrite
+this URL.
 
 Example Pillar
 ==============

bind/config.sls

@@ -130,7 +130,7 @@ bind_default_zones:
         map: {{ map }}
 {% endif %}
 
-{% for zone, zone_data in salt['pillar.get']('bind:configured_zones', {}).iteritems() -%}
+{% for zone, zone_data in salt['pillar.get']('bind:configured_zones', {}).items() -%}
 {%- set file = salt['pillar.get']("bind:available_zones:" + zone + ":file") %}
 {% if file and zone_data['type'] == "master" -%}
 zones-{{ zone }}:
@@ -157,8 +157,8 @@ signed-{{ zone }}:
 {% endif %}
 {% endfor %}
 
-{%- for view, view_data in salt['pillar.get']('bind:configured_views', {}).iteritems() %}
-{% for zone, zone_data in view_data.get('configured_zones', {}).iteritems() -%}
+{%- for view, view_data in salt['pillar.get']('bind:configured_views', {}).items() %}
+{% for zone, zone_data in view_data.get('configured_zones', {}).items() -%}
 {%- set file = salt['pillar.get']("bind:available_zones:" + zone + ":file") %}
 {% if file and zone_data['type'] == "master" -%}
 zones-{{ view }}-{{ zone }}:

bind/files/arch/named.conf

@@ -11,7 +11,7 @@ options {
     //  listen-on { none; };
 
     {#- Allow inclusion of arbitrary statements #}
-    {%- for statement, value in salt['pillar.get']('bind:config:options', {}).iteritems() -%}
+    {%- for statement, value in salt['pillar.get']('bind:config:options', {}).items() -%}
         {%- if value is iterable and value is not string %}
             {{ statement }} {
             {%- for item in value %}

bind/files/arch/named.conf.local

@@ -6,7 +6,7 @@
 // organization
 //include "/etc/bind/zones.rfc1918";
 
-{% for key,args in salt['pillar.get']('bind:configured_zones', {}).iteritems()  -%}
+{% for key,args in salt['pillar.get']('bind:configured_zones', {}).items()  -%}
 {%- set file = salt['pillar.get']("bind:available_zones:" + key + ":file") %}
 {%- set masters = salt['pillar.get']("bind:available_zones:" + key + ":masters") %}
 zone "{{ key }}" {

bind/files/debian/named.conf.key

@@ -3,7 +3,7 @@
 // Do any local configuration here
 //
 
-{% for key,args in salt['pillar.get']('bind:keys', {}).iteritems()  -%}
+{% for key,args in salt['pillar.get']('bind:keys', {}).items()  -%}
 key "{{ key }}" {
   algorithm {{ args['algorithm'] | default('HMAC-MD5.SIG-ALG.REG.INT') }};
   secret "{{ args['secret'] }}";

bind/files/debian/named.conf.local

@@ -60,13 +60,13 @@ zone "{{ key }}" {
 include "{{ map.default_zones_config }}";
 {%- endif %}
 
-{% for key, args in salt['pillar.get']('bind:configured_zones', {}).iteritems() -%}
+{% for key, args in salt['pillar.get']('bind:configured_zones', {}).items() -%}
 {%- set file = salt['pillar.get']("bind:available_zones:" + key + ":file") %}
 {%- set masters = salt['pillar.get']("bind:available_zones:" + key + ":masters") %}
 {{ zone(key, args, file, masters) }}
 {% endfor %}
 
-{% for view, view_data in salt['pillar.get']('bind:configured_views', {}).iteritems() %}
+{% for view, view_data in salt['pillar.get']('bind:configured_views', {}).items() %}
 
 view {{ view }} {
 {%- if view == 'default' %}
@@ -79,7 +79,7 @@ match-clients {
 {%- endfor %}
 };
 
-{% for key, args in view_data.get('configured_zones', {}).iteritems() -%}
+{% for key, args in view_data.get('configured_zones', {}).items() -%}
 {%- set file = salt['pillar.get']("bind:available_zones:" + key + ":file") %}
 {%- set masters = salt['pillar.get']("bind:available_zones:" + key + ":masters") %}
   {{ zone(key, args, file, masters) }}

bind/files/debian/named.conf.options

@@ -21,7 +21,7 @@ options {
 {%- endif -%}
 
 {#- Allow inclusion of arbitrary statements #}
-{%- for statement, value in salt['pillar.get']('bind:config:options', {}).iteritems() -%}
+{%- for statement, value in salt['pillar.get']('bind:config:options', {}).items() -%}
     {%- if value is iterable and value is not string %}
         {{ statement }} {
         {%- for item in value %}

bind/files/redhat/named.conf.local

@@ -6,7 +6,7 @@
 // organization
 //include "/etc/bind/zones.rfc1918";
 
-{% for key,args in salt['pillar.get']('bind:configured_zones', {}).iteritems()  -%}
+{% for key,args in salt['pillar.get']('bind:configured_zones', {}).items()  -%}
 {%- set file = salt['pillar.get']("bind:available_zones:" + key + ":file") %}
 {%- set masters = salt['pillar.get']("bind:available_zones:" + key + ":masters") %}
 zone "{{ key }}" {

bind/map.jinja

@@ -3,7 +3,7 @@
         'pkgs': ['bind9', 'bind9utils', 'dnssec-tools'],
         'service': 'bind9',
         'config_source_dir': 'bind/files/debian',
-        'zones_source_dir': '/srv/salt/zones',
+        'zones_source_dir': 'zones',
         'config': '/etc/bind/named.conf',
         'local_config': '/etc/bind/named.conf.local',
         'key_config': '/etc/bind/named.conf.key',
@@ -20,7 +20,7 @@
         'pkgs': ['bind'],
         'service': 'named',
         'config_source_dir': 'bind/files/redhat',
-        'zones_source_dir': '/srv/salt/zones',
+        'zones_source_dir': 'zones',
         'config': '/etc/named.conf',
         'local_config': '/etc/named.conf.local',
         'default_config': '/etc/sysconfig/named',

pillar.example

@@ -3,6 +3,9 @@ bind:
     pkgs:
       - bind
     service: named
+    # take zonefiles from `salt://bind/zonedata`
+    # instead of `salt://zones`:
+    zones_source_dir: bind/zonedata
 
 bind:
   config: