Bug 53785 - remove sambaBlockingLocks from UDM shares/share
remove sambaBlockingLocks from UDM shares/share
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Shares
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-4
Assigned To: Florian Best
Dirk Wiesenthal
https://git.knut.univention.de/univen...
:
: 52299 (view as bug list)
Depends on: 53711
Blocks:
  Show dependency treegraph
 
Reported: 2021-09-14 09:14 CEST by Florian Best
Modified: 2023-06-21 09:24 CEST (History)
5 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.017
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): API change, Cleanup
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2021-09-14 09:14:53 CEST
We should remove sambaBlockingLocks from UDM shares/share.

The property is not evaluated anymore since UCS 5.0. We still support mixed environments with UCS 4.4, therefore we don't remove it now.

management/univention-directory-manager-modules/modules/univention/admin/handlers/shares/share.py:mapping.register('sambaBlockingLocks', 'univentionShareSambaBlockingLocks', None, univention.admin.mapping.ListToString)
management/univention-directory-manager-modules/modules/univention/admin/handlers/shares/share.py:      'sambaBlockingLocks': univention.admin.property(
management/univention-directory-manager-modules/modules/univention/admin/handlers/shares/share.py:              ['sambaLocking', 'sambaBlockingLocks'],
management/univention-directory-manager-modules/modules/univention/admin/handlers/shares/share.py:mapping.register('sambaBlockingLocks', 'univentionShareSambaBlockingLocks', None, univention.admin.mapping.ListToString)

Adjust the test cases:
test/ucs-test/tests/53_samba-common/61_share_hacking:                   'univentionShareSambaBlockingLocks': ['1'],
test/ucs-test/tests/64_udm-shares/01_create_fileshare:                          'univentionShareSambaBlockingLocks': [properties['sambaBlockingLocks']],
test/ucs-test/tests/64_udm-shares/02_create_fileshare_and_connect_via_samba:                            'univentionShareSambaBlockingLocks': [properties['sambaBlockingLocks']],
test/ucs-test/tests/53_samba-common/10samba-share-options:sambaBlockingLocks/blocking locks/1
test/ucs-test/tests/64_udm-shares/01_create_fileshare:                  'sambaBlockingLocks': random.choice(['0', '1']),
test/ucs-test/tests/64_udm-shares/01_create_fileshare:                          'univentionShareSambaBlockingLocks': [properties['sambaBlockingLocks']],
test/ucs-test/tests/64_udm-shares/02_create_fileshare_and_connect_via_samba:                    'sambaBlockingLocks': random.choice(['0', '1']),
test/ucs-test/tests/64_udm-shares/02_create_fileshare_and_connect_via_samba:                            'univentionShareSambaBlockingLocks': [properties['sambaBlockingLocks']],

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

We removed the deprecated option "blocking locks" with #49898. 
After an update, this produces the following warning on each univention-s4search 
(or any other ldbadd, ldbsearch etc.)

"WARNING: The "blocking locks" option is deprecated"

This has no functional implications, but it lead some of our tests to fail and it might unsettle some costumers.
Comment 1 Florian Best univentionstaff 2022-04-25 18:09:11 CEST
*** Bug 52299 has been marked as a duplicate of this bug. ***
Comment 2 Florian Best univentionstaff 2022-04-26 18:21:43 CEST
The MR https://git.knut.univention.de/univention/ucs/-/merge_requests/360 at least removes the from being visible in the UI.
Comment 3 Florian Best univentionstaff 2023-06-09 11:41:46 CEST
The properties have been removed from the layout. It would be an API change to remove them completely, therefor postpone that to UCS 6.0.

univention-directory-manager-modules (15.0.20-9)
100917ad7edb | Bug #53785: remove checkbox "Blocking locks"
Comment 4 Dirk Wiesenthal univentionstaff 2023-06-19 10:38:46 CEST
Changelog: OK
Code review: OK
Tests: OK
Comment 5 Philipp Hahn univentionstaff 2023-06-21 09:24:42 CEST
UCS 5.0-4 has been released:
 https://docs.software-univention.de/release-notes/5.0-4/en/

If this error occurs again, please use the 'Clone This Bug' option.