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

Update alias.rst #13360

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update alias.rst #13360

wants to merge 1 commit into from

Conversation

stasic
Copy link
Contributor

@stasic stasic commented Oct 12, 2023

If a zone is operated with different nameserver software than PowerDNS and ALIAS is used it could result in an out-of-sync sencondary.

Short description

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)
  • checked that this code was merged to master

If a zone is operated with different nameserver software than PowerDNS and ALIAS is used it could result in an out-of-sync sencondary.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 6492285293

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 47 unchanged lines in 12 files lost coverage.
  • Overall coverage decreased (-0.02%) to 56.357%

Files with Coverage Reduction New Missed Lines %
pdns/sstuff.hh 1 60.24%
pdns/dbdnsseckeeper.cc 2 85.61%
pdns/iputils.hh 2 73.66%
pdns/misc.cc 2 61.32%
pdns/channel.hh 3 55.97%
pdns/dnsdistdist/dnsdist-healthchecks.cc 3 55.31%
pdns/stubresolver.cc 3 81.63%
pdns/tcpiohandler.cc 3 66.83%
pdns/test-distributor_hh.cc 3 88.66%
pdns/dnsdist.cc 5 71.05%
Totals Coverage Status
Change from base Build 6486035458: -0.02%
Covered Lines: 102357
Relevant Lines: 151170

💛 - Coveralls

@Habbie Habbie added this to the auth-4.9.0 milestone Oct 12, 2023
@klaus-nicat
Copy link
Contributor

I would add the sentence:
"To keep secondary nameservers up2date with the ALIAS target, you have to periodically retransfer the zone from PowerDNS, to force ALIAS-resolving during AXFR. This can be done either "dumb", for example every minute, or implement some "out of band" logic to retransfer zones only if the ALIAS target actually changed."

@Habbie Habbie modified the milestones: auth-4.9.0, auth-helpneeded Jan 23, 2024
Copy link
Member

@Habbie Habbie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and please look at Klaus' comment, thanks!

@@ -49,6 +49,8 @@ When set to 'yes', PowerDNS will look up the A and AAAA records of the name in t
This is useful when your secondary servers do not understand ALIAS, or should not look up the addresses themselves.
Note that secondaries will not automatically follow changes in those A/AAAA records unless you AXFR regularly.

If secondaries are operated with nameserver software which is not capable of synthesising ALIAS records the zone will probably go out of sync. Because updated ALIAS target records will not result in an increment of the Serial, which means that Secondaries won't pull the Zone.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
If secondaries are operated with nameserver software which is not capable of synthesising ALIAS records the zone will probably go out of sync. Because updated ALIAS target records will not result in an increment of the Serial, which means that Secondaries won't pull the Zone.
If secondaries are operated with name server software which is not capable of synthesizing ALIAS records the zone will probably go out of sync. Because updated ALIAS target records will not result in an increment of the Serial, which means that Secondaries won't pull the Zone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants