Changed slave zone handling from dns module.

dns/manifests/init.pp

@@ -183,13 +183,16 @@ class dns::server {
 # $role:
 #       The role {master, slave} of this host.
 # $master:
-#      IP address and FQDN or hostname of the DNS master for this zone.
+#      IP address of DNS master for this zone if role is slave.
+#      IP address and FQDN of DNS master for this zone if running as
+#      master and using autogenerated zone.
 # $slaves:
 #      IP addresess and host names of the DNS slaves for this zone.
+#      Required only when using autogenrated zones.
 # $source:
 #      Source file to use for zone. Defaults to auto.
 #
-define dns::zone($role = "master", $master = [], $slaves = [], $source = "AUTO") {
+define dns::zone($role = "master", $master = "", $slaves = [], $source = "AUTO") {
 
     $zone = $name
     case $role {
@@ -207,6 +210,9 @@ define dns::zone($role = "master", $master = [], $slaves = [], $source = "AUTO")
             }
         }
         "slave": {
+            if $master == "" {
+                fail("No master defined for dns::zone '${name}'")
+            }
             case $operatingsystem {
                 "openbsd": {
                     $zonedir = "/slave"

dns/templates/zone.slave.erb

@@ -2,11 +2,9 @@ zone "<%= zone %>" {
     type slave;
 <% if zone.match(/\//) %>
     file "<%= zonedir %>/db.<%= zone.sub(/\//, '-') %>";
-    <% else %>
+<% else -%>
     file "<%= zonedir %>/db.<%= zone %>";
 <% end -%>
-    <% master.each_pair do |k, v| -%>
+    masters { <%= master %>; };
-    masters { <%= v['ip'] %>; };
-    <% end -%>
     allow-transfer { none; };
 };