Bug 46578 - Uninstalling the App uvmm fails
Uninstalling the App uvmm fails
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Virtualization - UVMM
UCS 4.3
Other Linux
: P5 normal (vote)
: UCS 4.3
Assigned To: Stefan Gohmann
Dirk Wiesenthal
: interim-4
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2018-03-10 10:32 CET by Dirk Wiesenthal
Modified: 2018-03-14 14:38 CET (History)
2 users (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 Dirk Wiesenthal univentionstaff 2018-03-10 10:32:43 CET
From join.log on a DC Slave:

RUNNING 56univention-virtual-machine-manager-daemon.uinst
Object modified: cn=slave105,cn=dc,cn=computers,dc=autotest105,dc=local
Object removed: cn=Virtual Machine Manager,cn=services,cn=univention,dc=autotest105,dc=local
Missing LDAP credentials
EXITCODE=1

We cannot ship UCS 4.3 with the App as long as this is not fixed. Maybe only "$@" is missing?
Comment 1 Dirk Wiesenthal univentionstaff 2018-03-10 10:33:47 CET
Setting to interim-4. Is the App blocking the release?
Comment 2 Stefan Gohmann univentionstaff 2018-03-10 21:40:38 CET
Do you know which parameter are given to the uinst script? How do you call the uninstallation?
Comment 3 Dirk Wiesenthal univentionstaff 2018-03-11 08:54:39 CET
Sorry, this is not uvmm-ec2 but uvmm itself (the test uninstalls this "dependency App", too). And the App Center calls univention-run-join-scripts correctly. The unjoin script
/usr/lib/univention-install/56univention-virtual-machine-manager-daemon.uinst
is just buggy.

It parses the --binddn and --bindpwd itself, but does not pass "$@" to that parse function. Therefore it fails. I have edited the file locally and then it works.
Comment 4 Stefan Gohmann univentionstaff 2018-03-11 12:38:36 CET
Ah, you mean in this way?

[4.3-0 c4aae89ca0] * 56univention-virtual-machine-manager-daemon.uinst: Pass "$@" to the   credential parsing functions (Bug #46578)


[4.3-0 fb489c5c2c] Changelog Bug #46578
Comment 5 Dirk Wiesenthal univentionstaff 2018-03-11 18:08:59 CET
(In reply to Stefan Gohmann from comment #4)
> Ah, you mean in this way?
Tests say yes. Changelog okay.
Comment 6 Stefan Gohmann univentionstaff 2018-03-14 14:38:19 CET
UCS 4.3 has been released:
 https://docs.software-univention.de/release-notes-4.3-0-en.html
 https://docs.software-univention.de/release-notes-4.3-0-de.html

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