samba_dnsupdate: honor 'dns zone scavenging' option, only update if needed
commit9bb128fc78527d77ddabc4c4dc21f77788b6cc22
authorBjörn Baumbach <bb@sernet.de>
Wed, 5 Sep 2018 14:54:01 +0000 (5 16:54 +0200)
committerKarolin Seeger <kseeger@samba.org>
Thu, 13 Sep 2018 06:38:16 +0000 (13 08:38 +0200)
tree52cb3407963a22348e97b00f05164ebdbcbae0b7
parentb94c676eb1a8e50255cf5e23c8178bcf1270e8d2
samba_dnsupdate: honor 'dns zone scavenging' option, only update if needed

Since scavenging is implemented the samba_dnsupdate command always updates all
dns records required by the dc. This is not needed if dns zone scavenging
is not enabled.

This avoids the repeating TSIG error messages:

 # samba_dnsupdate --option='dns zone scavenging = yes' 2>&1 | uniq -c
     29 ; TSIG error with server: tsig verify failure
      1 Failed update of 29 entries
 # echo ${PIPESTATUS[0]}
 29

 # samba_dnsupdate --option='dns zone scavenging = no' 2>&1 | uniq -c
 # echo ${PIPESTATUS[0]}
 0

Note that this results in about 60 lines in the log file,
which triggered every 10 minutes ("dnsupdate:name interval=600" is the default).

This restores the behavior before 8ef42d4dab4dfaf5ad225b33f7748914f14dcd8c,
if "dns zone scavenging" is not switched on (which is still the default).

Avoiding the message from happening at all is subject for more debugging,
most likely they are caused by bugs in 'nsupdate -g' (from the bind package).

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13605

Pair-programmed-with: Stefan Metzmacher <metze@samba.org>

Signed-off-by: Björn Baumbach <bb@sernet.de>
Signed-off-by: Stefan Metzmacher <metze@samba.org>
selftest/knownfail.d/dns
source4/scripting/bin/samba_dnsupdate