Any way to separate the real zones from the bind::server::conf class? about puppet-bind

[pablokbs]

Thanks, but that's not what I'm asking, I need to have only simple line on site.pp for each domain, I don't want to paste all that code there. That's why I'm creating a "define"

Also, even if I paste that at the beggining of the site.pp, I still need to add the zone to the zone hash, I mean, to add a domain I have to add it in two places, that's what I want to avoid.

[b4ldr]

@pablokbs last one wouldn't work so i deleted it something like this should work though (untested and probably syntax spelling errors but you should get the idea)

site.pp

  $domains      => {
    example.com => { domain_ip => '10.0.0.1', spf => true, dkim => true, mx => "mx" },
    example.org => { domain_ip => '10.0.0.1', spf => true, dkim => true, mx => "mx" }
  }
  class {'::custom_bind':
    domains => $domains
  }

custom_bind/init.pp

class custom_bind ($domains) {
  include bind
  bind::server::conf { '/etc/named.conf':
    listen_on_addr    => [ 'any' ],
    listen_on_v6_addr => [ 'any' ],
    forwarders        => [ '8.8.8.8', '8.8.4.4' ],
    allow_query       => [ 'localnets' ],
    includes          => [ '/etc/named/zones.conf' ]
  }

  create_resources(custom_bind::domain, $domains)
  file {'/etc/named/zones.conf':
    ensure => present,
    content  => template('bind/zones.conf.erb'),
  }
}

custom_bind/domain.pp

define custom_bind::domain ($domain_ip, $spf, $dkim, $mx) {
  bind::server::file { "${name}.zone":
    zone_type => "master",
    content  => template('bind/zone.erb'),
  }
}

zone.conf.erb

<% @domains.each_key do |zone| -%>
zone <%= zone %> {
  type master;
  file <%= zone %>;
};

[pablokbs]

I had to make some adjustments but it worked perfectly, thank you very much.