Bug 52745 - 01_base/96rename* fail in UCS5
01_base/96rename* fail in UCS5
Status: CLOSED FIXED
Product: UCS Test
Classification: Unclassified
Component: General
unspecified
Other Linux
: P5 normal (vote)
: UCS 5.0
Assigned To: Julia Bremer
Florian Best
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2021-02-05 09:42 CET by Julia Bremer
Modified: 2021-05-25 16:02 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Development Internal
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

Note You need to log in before you can comment on or make changes to this bug.
Description Julia Bremer univentionstaff 2021-02-05 09:42:40 CET
In UCS5 all the testcases which test the renaming of well-known groups and users (Administrator, Domain Admins, Domain Users) fail almost constantly. 
They rename the user/group, wait for replication and postrun, check if the corresponding ucr var is set, and check all the ucr templates, if those well-known objects are renamed in them.

In scenarios with S4, usually the postrun which sets the UCR var 
(groups/default/domainadmins/ groups/default/domainusers /users/default/Administrator)
is never executed, and the test fails with "LDAPReplication Failed".
Every time I observed this, an S4Connector loop was present (See Bug #52358).
This loop most likely prevented the postrun. 

In scenarios without S4, 96rename_domain_admins and 95rename_administrator fail, even though the postrun is executed and the UCR var is renamed. 
Still, some of the templates are not renamed when the test checks them.
On my machine, it was necessary to increase some "sleeps" for the tests to be successful.

I find it puzzling though, why it seems to take quite a bit longer than in UCS4, where the tests always succeed.
Comment 1 Julia Bremer univentionstaff 2021-02-12 08:51:41 CET
4817775503 Bug #52745: Put some files on ignorelist, wait longer
58c2e7e361 Bug #52745: version bump
8a55e8a829 Bug #52745: use new admin credentials for cleanup
ca03aa0710 Bug #52745: Translate 95rename_administrator to python

I adjusted the tests, but some still failed due to Bug #34003 (it took a long time to commit slapd.conf)
And some failed due to Bug #52358 (S4Connector loop inhibited postrun).
Fixes on those bugs repaired these tests.
Comment 2 Florian Best univentionstaff 2021-03-10 11:21:58 CET
as discussed, please adjust the test cases according to:
85605da9f0309ef654d7e4991b5c8a34fe95e945 + ac46d66d9fb1c759faa41d06dd32fbc5f4f35be7
Comment 3 Julia Bremer univentionstaff 2021-03-23 14:06:21 CET
f3ac77b942 Bug #52745: Code cleanup
a9d48c25bd Bug #50278: fixup! Code cleanup

done
Comment 4 Florian Best univentionstaff 2021-03-23 14:13:00 CET
-                               wait_for_drs_replication('(sAMAccountName=%s)' % (new_admin_name,))
+                               wait_for_drs_replication(filter_format('(sAMAccountName=%s)' % (new_admin_name,)))

→ "%" has to be replaced with "," (2 times)
Comment 5 Julia Bremer univentionstaff 2021-03-23 14:33:03 CET
(In reply to Julia Bremer from comment #3)

> a9d48c25bd Bug #50278: fixup! Code cleanup
> 

did that in this commit
Comment 6 Florian Best univentionstaff 2021-03-23 14:43:32 CET
alright.
Comment 7 Florian Best univentionstaff 2021-05-25 16:02:42 CEST
UCS 5.0 has been released:
 https://docs.software-univention.de/release-notes-5.0-0-en.html
 https://docs.software-univention.de/release-notes-5.0-0-de.html

If this error occurs again, please use "Clone This Bug".