Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SOA has additional "." #93

Closed
eLvErDe opened this issue Apr 3, 2015 · 3 comments
Closed

SOA has additional "." #93

eLvErDe opened this issue Apr 3, 2015 · 3 comments

Comments

@eLvErDe
Copy link

eLvErDe commented Apr 3, 2015

Hi,

With this config:

dns::zone { 'voice.domain.com': zone_type => 'master', nameservers => [ 'nsc-srva', 'rpi' ], }
dns::record::a {
  'nsc-srva.voice.domain.com': zone => 'voice.domain.com', data => [ $srva_ip_addr ];
  'rpi.voice.domain.com':      zone => 'voice.domain.com', data => [ $rpi_ip_addr  ];
  '.voice.domain.com':         zone => 'voice.domain.com', data => [ $voice_ip_addr ]; 
}

The generated zones looks like this (note the SOA line with double dots). I guess it's a bug, as I haven't set the soa params so it uses default values!

root@rpi:~# cat /etc/bind/zones/db.voice.domain.com
;
; BIND data file for voice.domain.com zone.
; File managed by puppet.
;
$ORIGIN voice.domain.com.
$TTL    604800
@   IN  SOA rpi.nrack01.domain.com.. root.rpi.nrack01.domain.com.. (
[...]

Best regards, Adam.

@solarkennedy
Copy link
Collaborator

Hmmm. Can you PR to fix this issue?

@ajjahn
Copy link
Owner

ajjahn commented Apr 17, 2015

Looks like a dot is being appended to $soa here as well as here. @solarkennedy Can you think of any reason it should be appended in zone.pp vs the zone_file template?

@solarkennedy
Copy link
Collaborator

Um, I'm not really sure.
On the one hand, it seems like the module should be adding in the dot somewhere, but on the other hand I like it when there is no extra magic.

More senior dns people will be adding the dots at the end of their entries out of habit, leading to the double dot, to me this is surprising.

I vote we leave the dot appended in the class variable, no extra help except other than a good default. (in other words, no hard coded extra dot in the template)

oloc added a commit to oloc/puppet-dns that referenced this issue May 18, 2015
Fixes ajjahn#93 - Avoid the extra dot in the soa
oloc added a commit to oloc/puppet-dns that referenced this issue May 18, 2015
solarkennedy added a commit that referenced this issue May 25, 2015
Fixes #93 - Avoid the extra dot in the soa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants