Bug 31362 - direct use of UMC API
direct use of UMC API
Status: RESOLVED WONTFIX
Product: UCS extended documentation
Classification: Unclassified
Component: Developer documentation
unspecified
Other Linux
: P5 normal (vote)
: ---
Assigned To: Docu maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-15 10:49 CEST by Philipp Hahn
Modified: 2024-04-17 13:19 CEST (History)
1 user (show)

See Also:
What kind of report is it?: ---
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.
Comment 1 Florian Best univentionstaff 2021-10-09 12:56:14 CEST
umc-command, umc-client should be removed - not documented.
Comment 2 Philipp Hahn univentionstaff 2021-10-09 13:02:13 CEST
(In reply to Florian Best from comment #1)
> umc-command, umc-client should be removed - not documented.

These tools are currently used by
- management/univention-management-console-module-udm/umc/test-commands
- test/ucs-test/tests/
- base/univention-updater/tests/test_umc-commands.sh

So they SHOULD be documented UNLESS you name a replacement CLI tool INCLUDING documentation for it!
Comment 3 Philipp Hahn univentionstaff 2021-10-09 13:04:02 CEST
(In reply to Philipp Hahn from comment #2)
> (In reply to Florian Best from comment #1)
> > umc-command, umc-client should be removed - not documented.
> 
> These tools are currently used by
> - management/univention-management-console-module-udm/umc/test-commands
> - test/ucs-test/tests/
> - base/univention-updater/tests/test_umc-commands.sh

# git grep -n -e umc-client -e umc-command -- ':!**/debian/*' | wc -l
46
Comment 4 Florian Best univentionstaff 2021-10-09 14:15:45 CEST
(In reply to Philipp Hahn from comment #2)
> (In reply to Florian Best from comment #1)
> > umc-command, umc-client should be removed - not documented.
> 
> These tools are currently used by
> - management/univention-management-console-module-udm/umc/test-commands
> - test/ucs-test/tests/
> - base/univention-updater/tests/test_umc-commands.sh
can be dropped.
The only relevant one which we should and need to replace are:
base/univention-system-setup/usr/lib/univention-system-setup/cleanup-post.d/08_activate_license:        /usr/sbin/umc-command -r -U "$hostname\$" -y /etc/machine.secret udm/request_new_license -f license-request -o email="$email"
services/univention-samba/scripts/addmachine.sh:/usr/sbin/umc-command -s "$ldap_master" -y /etc/machine.secret -U "$hostname$" selectiveudm/create_windows_computer -o name="$name" -o samba3_mode=True; rc=$

> So they SHOULD be documented UNLESS you name a replacement CLI tool
> INCLUDING documentation for it!
Here is a replacement which doesn't use UMCP but univention.lib.umc.Client:

https://git.knut.univention.de/univention/ucs/-/blob/fbest/43633-unify-umc-server/management/univention-management-console/scripts/univention-management-console-client
Comment 5 Nico Gulden univentionstaff 2024-04-17 13:19:06 CEST
This bug hasn't seen any update for several years. I close it.

If you still see a need for it, you can reopen the bug. Please add an argumentation about why it's important to take care of it.