general cleanup

defaults/main.yml

@@ -20,4 +20,14 @@ powerdns:
     "tcp-control-secret": "{{ lookup('password', '/dev/null length=64') }}"
     "version-string": "1"
   zonemeta: {}
+  zonemeta_defaults:
+    type: master
+    "allow-query":
+    - any
+    "allow-update":
+    - none
+    "allow-transfer":
+    - none
+    "slaves":
+    - none
   zones: {}

handlers/main.yml

@@ -1,3 +1,6 @@
+- name: delete pdns bind.conf
+  file: path=/etc/powerdns/pdns.d/bind.conf state=absent
+
 - name: copy bind zone file
   copy:
     src: "/var/lib/powerdns/tpl/{{ item.item }}"

tasks/main.yml

@@ -4,6 +4,8 @@
     pkg:
       - pdns-server
       - pdns-backend-bind
+  notify:
+  - delete pdns bind.conf
 
 - name: delete debian defaults
   file:

templates/backend-bind.conf.j2

@@ -3,25 +3,27 @@ options {
 };
 
 {% for z in powerdns.zonemeta %}
-zone "{{ z }}" IN {
+{% set zonename = z %}
-        type {{ powerdns.zonemeta[z].type }};
+{% set zone = {}|combine(powerdns.zonemeta_defaults, powerdns.zonemeta[zonename]) %}
-        file "/var/lib/powerdns/zones/db.{{z}}";
+zone "{{ zonename }}" IN {
-{%if 'masters' in powerdns.zonemeta[z] %}
+        type {{ zone.type }};
+        file "/var/lib/powerdns/zones/db.{{ zonename }}";
+{% if 'masters' in zone %}
 	masters { 
-		{{ powerdns.zonemeta[z]['masters']|join('; ') }}; 
+		{{ zone.masters|join('; ') }};
 	};
 {% endif %}
         allow-query { 
-		{{ powerdns.zonemeta[z]['allow-query']|default(['any'])|join('; ') }}; 
+		{{ zone['allow-query']|join('; ') }};
 	};
         allow-update { 
-		{{ powerdns.zonemeta[z]['allow-update']|default(['none'])|join('; ') }}; 
+		{{ zone['allow-update']|join('; ') }};
 	};
         allow-transfer { 
-		{{ powerdns.zonemeta[z]['allow-transfer']|default(['none'])|join('; ') }}; 
+		{{ zone['allow-transfer']|join('; ') }};
 	};
-{%if 'notify' in powerdns.zonemeta[z] %}
+{%if 'notify' in zone %}
-	notify {{ "yes" if powerdns.zonemeta[z].notify else "no" }};
+	notify {{ "yes" if zone.notify else "no" }};
 {% endif %}
 };