Bug 58787 - Support move during modify (sync, read), changes in AD
Summary: Support move during modify (sync, read), changes in AD
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: AD Connector
Version: UCS 5.0
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 5.0-10-errata
Assignee: Felix Botner
QA Contact: Jürn Brodersen
URL: https://git.knut.univention.de/univen...
Keywords:
Depends on:
Blocks: 58793
  Show dependency tree / graph
 
Reported: 2025-11-06 16:42 CET by Felix Botner
Modified: 2025-11-12 13:29 CET (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):
Customer ID:
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 Felix Botner univentionstaff 2025-11-06 16:42:02 CET 
Some operations in AD lead to a move of the object in UCS, we need to support this in sync_to_ucs::modify.
Comment 1 Felix Botner univentionstaff 2025-11-06 16:45:32 CET 
Also https://git.knut.univention.de/univention/dev/internal/dev-issues/dev-incidents/-/issues/175 (uniqueMember: value #0 already exists when renaming with umlauts)
Comment 2 Felix Botner univentionstaff 2025-11-07 15:49:40 CET 
https://git.knut.univention.de/univention/dev/internal/dev-issues/dev-incidents/-/issues/175:
f3e6c0247c1 chore(ad-connector): update advisory
59ed1954da1 test(ucs-test): test_rename_user_with_umlauts_in_ad
6a126f8e75e fix(ad-connector): compare strings in__compare_lowercase*

https://git.knut.univention.de/univention/dev/internal/dev-issues/dev-incidents/-/issues/181
06482aa20b1 chore(ad-connector): update advisory
0203717ab59 test(ucs-test): 504test_group_cache_after_move.py add mixed case
ca8775b1265 chore(ucs-test): debian/changelog
3bfa2250161 chore(ad-connector): debian/changelog
3028125194b test(ucs-test): 504test_group_cache_after_move.py
503c1d5e4e8 fix(ad-connector): update group cache after move/rename

Successful build
Package: ucs-test
Version: 10.0.24-33
Release: 5.0-0
Scope: errata5.0-10

Successful build
Package: univention-ad-connector
Version: 14.0.28
Release: 5.0-0
Scope: errata5.0-10
Comment 3 Felix Botner univentionstaff 2025-11-11 09:56:28 CET 
https://git.knut.univention.de/univention/dev/ucs/-/issues/3222
d300905f661 chore(ad-connector): update advisory
51f4972dd8a chore(ad-connector): changelog
d03beb9de1c chore(ucs-test): changelog
31ad7cdbf07 test(ucs-test): test_rename_ou_rename_group
bc36b29c6cc fix(ad-connector): ignore order when checkig for changed attributes
098f8a7fd99 test(ucs-test): 402sync_test_ignore_attribute_performance
c1f832b1dc2 fix(ad-connector): ingnore order when checkig for changed attributes

Successful build
Package: ucs-test
Version: 10.0.24-34
Release: 5.0-0
Scope: errata5.0-10

Successful build
Package: univention-ad-connector
Version: 14.0.29
Release: 5.0-0
Scope: errata5.0-10
Comment 4 Jürn Brodersen univentionstaff 2025-11-12 12:05:14 CET 
upgrade ok
yaml ok
jenkins ok
smoke test with sync/read mode (create & change user) ok

-> verified