Bug 40343

Summary: Press reboot and the message "The update is being executed" is being shown
Product: UCS Reporter: Stefan Gohmann <gohmann>
Component: UMC - Software updateAssignee: Florian Best <best>
Status: CLOSED FIXED QA Contact: Stefan Gohmann <gohmann>
Severity: normal    
Priority: P5 CC: best, walkenhorst
Version: UCS 4.1   
Target Milestone: UCS 4.1-0-errata   
Hardware: Other   
OS: Linux   
See Also: https://forge.univention.org/bugzilla/show_bug.cgi?id=26544
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: updater-message.png
patch

Description Stefan Gohmann univentionstaff 2015-12-28 12:44:02 CET
Created attachment 7387 [details]
updater-message.png

I've updated from UCS 4.0-4 to UCS 4.1-0. Afterwards, the updater modules shows the information that the system has to be rebooted. I clicked on the reboot button and after about 10 seconds, the attached message was shown: "The update is being executed ...".
Comment 1 Florian Best univentionstaff 2015-12-28 15:56:36 CET
We should use umc.modules.lib.askReboot() instead which shows a progressbar during the reboot.
And the custom error handling should be replaced by the generic one from UMC.
If we replace the form by direct tools.umcpCommand calls this would be easier → Bug #26544.
Comment 2 Florian Best univentionstaff 2016-01-15 11:57:56 CET
Created attachment 7410 [details]
patch

And here is the patch which displays a progressbar during reboot.
Comment 3 Florian Best univentionstaff 2016-01-15 12:52:27 CET
univention-updater (11.0.7-12):
r66819 | Bug #40343: display progressbar during system reboot

univention-updater.yaml:
r66820 | YAML Bug #40343
Comment 4 Stefan Gohmann univentionstaff 2016-01-20 06:53:40 CET
YAML: OK (minor adjustments r66893)

Code review: OK

Tests: OK, it works and looks much better.
Comment 5 Janek Walkenhorst univentionstaff 2016-01-20 13:24:51 CET
<http://errata.software-univention.de/ucs/4.1/58.html>