Bug 53623 - Synchronisation of DNS TTL data
Synchronisation of DNS TTL data
Status: NEW
Product: UCS
Classification: Unclassified
Component: S4 Connector
UCS 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: Samba maintainers
Samba maintainers
https://docs.microsoft.com/en-us/open...
:
Depends on: 23732
Blocks:
  Show dependency treegraph
 
Reported: 2021-07-29 17:05 CEST by Dirk Schnick
Modified: 2021-08-06 09:02 CEST (History)
4 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.114
Enterprise Customer affected?: Yes
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2021072821000259
Bug group (optional):
Max CVSS v3 score:


Attachments
bug53623_incomplete_draft.patch (8.60 KB, patch)
2021-08-03 13:52 CEST, Arvid Requate
Details | Diff
bug53623.patch (10.79 KB, patch)
2021-08-03 18:10 CEST, Arvid Requate
Details | Diff
bug53623-ucs448.patch (11.17 KB, patch)
2021-08-05 12:44 CEST, Arvid Requate
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schnick univentionstaff 2021-07-29 17:05:43 CEST
As customer requested I will reopen this bug and set the version up to UCS 4.4; I'm sure the problem is still relevant in UCS 5. The ttl is not synchronized to samba.


+++ This bug was initially created as a clone of Bug #23732 +++

Die DNS TTL Daten werden im Moment nicht synchronisiert. Ich denke es wäre gut, wenn wir das noch zu 3.0 umsetzen.
Comment 1 Dirk Schnick univentionstaff 2021-08-03 07:59:48 CEST
To verify the problem easily change the TTL in UMC (actually you do not need to do that as UMC default TTL is 3 hours) in a /dns/backend=samba domain to a different value than 900 minutes. This change will never applied.

A fast rgrep through our s4 code shows that we sync hardcoded 900 minute ttl:

rgrep ttl=900
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:# def __init__(self, ip_addr, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:# def __init__(self, ip6_addr, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:# def __init__(self, dns_server, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:# def __init__(self, target, port, priority=0, weight=100, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:# def __init__(self, cname, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:# def __init__(self, slist, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:	def __init__(self, ptr, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):
services/univention-s4-connector/modules/univention/s4connector/s4/dns.py:	def __init__(self, name, priority, serial=1, ttl=900, rank=dnsp.DNS_RANK_ZONE):


btw this code has not changed in UCS5 and so the bug will also apply to UCS5
Comment 2 Arvid Requate univentionstaff 2021-08-03 13:52:36 CEST
Created attachment 10791 [details]
bug53623_incomplete_draft.patch
Comment 3 Arvid Requate univentionstaff 2021-08-03 18:10:56 CEST
Created attachment 10792 [details]
bug53623.patch

This patch should be better.
Comment 4 Arvid Requate univentionstaff 2021-08-05 12:44:13 CEST
Created attachment 10794 [details]
bug53623-ucs448.patch
Comment 5 Florian Best univentionstaff 2021-08-05 16:19:00 CEST
A git branch with the patch would be nice. Maybe with a merge request, so that one can review it professionally.
Comment 6 Dirk Schnick univentionstaff 2021-08-06 09:02:53 CEST
Patch worked in my 448 testenvironment. Many thanks.