Bug 53731 – ignore cn=temporary,cn=univention DN's in S4-Connector

Bug 53731 - ignore cn=temporary,cn=univention DN's in S4-Connector


Summary:	ignore cn=temporary,cn=univention DN's in S4-Connector

Status:	NEW

Product:	UCS
Classification:	Unclassified
Component:	S4 Connector
Version:	UCS 5.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	Samba maintainers
QA Contact:	Samba maintainers

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2021-08-31 23:59 CEST by Florian Best
Modified:	2021-09-22 12:55 CEST (History)
CC List:	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?:	5: Blocking further progress on the daily work
User Pain:	0.286
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:	2021092221000186
Bug group (optional):	Troubleshooting, UCS Performance
Max CVSS v3 score:

Flags:	best: Patch_Available+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Florian Best

2021-08-31 23:59:13 CEST

We should ignore lock objects in the S4-Connector listener. These objects aren't consumed by the S4-Connector and trigger a useless transaction.

Patch:
diff --git services/univention-s4-connector/s4-connector.py services/univention-s4-connector/s4-connector.py
index 469aed304f..347f11307b 100644
--- services/univention-s4-connector/s4-connector.py
+++ services/univention-s4-connector/s4-connector.py
@@ -47,7 +47,7 @@ except ImportError:
 
 name = 's4-connector'
 description = 'S4 Connector replication'
-filter = '(objectClass=*)'
+filter = '(&(objectClass=*)(!(objectClass=lock)))'
 attributes = []  # type: List[str]
 
 # use the modrdn listener extension

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

With bug #48427 we are starting to store the transactions in a LDAP database. To reduce the number of write operations and the amount of disk space needed for the LDAP translog database, we should think about ignoring the temporary objects (cn=temporary,cn=univention) in the translog overlay.

Comment 1 Florian Best

2021-09-01 00:02:13 CEST

Probably even:

filter = '(&(objectClass=*)(!(objectClass=lock))(!(univentionLastUsedValue=*)))'

to ignore all temporary objects.