Bug 50905 – Reduce ldap load from ldap-group-to-file.py

Bug 50905 - Reduce ldap load from ldap-group-to-file.py


Summary:	Reduce ldap load from ldap-group-to-file.py

Status:	CLOSED FIXED

Product:	UCS@school
Classification:	Unclassified
Component:	General
Version:	UCS@school 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS@school 4.4 v5-errata
Assigned To:	Jürn Brodersen
QA Contact:	Ole Schwiegert

URL:
Keywords:

Depends on:	50904
Blocks:
	Show dependency tree / graph

Reported:	2020-03-09 10:45 CET by Jürn Brodersen
Modified:	2020-03-25 16:09 CET (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Feature Request
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

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 Jürn Brodersen

2020-03-09 10:45:23 CET

For @school we want to change the check_members default to not check the members not only for new installations but also for old existing setup.

The meta packages seems to be a good place for it

Comment 1 Ole Schwiegert

2020-03-09 13:58:16 CET

UCRV unset: OK
Branch test looks good: OK

I would merge the change to the postinst into the master.
I dont know what to do yet about test as we cannot merge it into the master like that.

Comment 2 Jürn Brodersen

2020-03-09 23:30:19 CET

I'm not sure about tests.
01_base/52proofuniquemember and 01_base/52proofuniquemember_empty_attribute do repair groups with members that don't exist. These test don't have the apptest tag so they aren't executed in our tests, though.

Adding a group in the scenario like I did in the branch test, feels quite a bit hacky. Any other ideas? Or is the branch test scenario sufficient?

Comment 3 Jürn Brodersen

2020-03-11 10:23:25 CET

Ok merged and build

Comment 4 Ole Schwiegert

2020-03-11 10:25:27 CET

Package installs: OK
Advisory: OK
Changes take effect: OK

Comment 5 Sönke Schwardt-Krummrich

2020-03-25 16:09:33 CET

UCS@school 4.4 v5 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.4v5-de.html

If this error occurs again, please clone this bug.