Backport the DNS code from 1.3-dev for speed and usability reason

Description

backport the dns code from 1.3-dev for speed and usability reason (see #5670 (closed)). The code is too slow to be really be usable in a real case scenario

Source FusionDirectory Version

1.3-dev

Destination FusionDirectory Version

1.2-fixes

Plugin with the new code

Dns

Reason of Backporting code

speed reason, too slow to be used in real world case, render plugin barely usable in zone editing and removal

What defect does it correct

Speed up deletion and saving by only touching record needed to be affected in place of the whole

Additional Information

Critical on certain customers site, OpenSides, Inalco

moved from fd#5687 (closed)

created branch 5688-backport-the-dns-code-from-1-3-dev-for-speed-and-usability-reason

assigned to @MCMic

changed the description

changed milestone to %FusionDirectory 1.2.1

added ~233 ~105 labels

added To Be Tested label

assigned to @bmortier and unassigned @MCMic

added PJ1610-0019 label

added Silver label

changed milestone to %50

added plugin-dns label

Can you tell me what changes are made because today it's still a bit slow.

If I check the ldap logs Fusion still iterates over all records :

-> This is a slapd log on the 1.3-dev where I modified a cname for test4

Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=727 MOD attr=objectClass zoneName dNSClass pTRRecord relativeDomainName
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=727 RESULT tag=103 err=0 text=
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=728 MOD dn="relativeDomainName=40,zoneName=72.10.10.in-addr.arpa.,zoneName=inalco.fr.,ou=dns,dc=example,dc=com"
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=728 MOD attr=objectClass zoneName dNSClass pTRRecord relativeDomainName
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=728 RESULT tag=103 err=0 text=
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=729 MOD dn="relativeDomainName=30,zoneName=72.10.10.in-addr.arpa.,zoneName=inalco.fr.,ou=dns,dc=example,dc=com"
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=729 MOD attr=objectClass zoneName dNSClass pTRRecord relativeDomainName
Oct 18 11:04:53 ldapt-m slapd[1648]: conn=4839 op=729 RESULT tag=103 err=0 text=

-> Same behavior on the 1.2-fixes

I though you added the ability to track only the changes made in the edition window.

The speed is clearly improved with the patch you provided, but as the number of records growed up when switching to production, it remains a little bit slow.

hello,

please use markdown to format correctly your logs, because its not readable

Cheers

updated

assigned to @MCMic and unassigned @bmortier

Hello,

please open a new bug with

• number of records
• numbers of zones
• timing of modifiying and saving an entry
• logging from ldap

and any other data you find usefull

Cheers

added 10m of time spent

mentioned in issue #5716 (closed)