Bug 36939 - Only execute setup scripts belonging to the flavor
Only execute setup scripts belonging to the flavor
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Basic settings
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 4.0-0-errata
Assigned To: Florian Best
Alexander Kläser
:
: 36941 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-21 17:57 CET by Florian Best
Modified: 2014-12-04 12:22 CET (History)
2 users (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:
best: Patch_Available+


Attachments
patch (2.48 KB, patch)
2014-11-21 17:57 CET, Florian Best
Details | Diff
Screenshot network settings page after applying the values has finished (33.55 KB, image/png)
2014-11-28 16:18 CET, Alexander Kläser
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2014-11-21 17:57:02 CET
Created attachment 6459 [details]
patch

Currently when saving the Network settings, language settings, or certificate settings all system setup scripts are executed. This has several drawbacks:
* progressbar messages are wrong
* it is slow
* some scripts are preventing a successful saving e.g. because no hostname is provided

Only the scripts belonging to the flavor should be executed.
Attached is a patch which does that.
Comment 1 Florian Best univentionstaff 2014-11-26 14:44:51 CET
patch has been applied.
Comment 2 Alexander Kläser univentionstaff 2014-11-28 16:18:44 CET
Created attachment 6482 [details]
Screenshot network settings page after applying the values has finished

If I see correctly, the progress bar needs to be adapted in order to account for the correct number of scripts being executed (see screenshot).

I spotted a minor thing, in setup.log I could not spot a __NAME___ line for the 10interfaces script. The patch bellow will fix that. Could you please apply it?

--- usr/lib/univention-system-setup/scripts/30_net/10interfaces (Revision 56301)
+++ usr/lib/univention-system-setup/scripts/30_net/10interfaces (Arbeitskopie)
@@ -44,6 +44,7 @@
 
 
 class NetworkSetup(SetupScript):
+       name = _("Configuring IPvX network interfaces")
 
        def __init__(self):
                super(NetworkSetup, self).__init__()
@@ -55,7 +56,6 @@
 
        def inner_run(self):
                self.parse_options()
-               self.name = _("Configuring IPvX network interfaces")
                self.setup_logging()
                self.setup_ud_logging()
                self.prepare_changeset()
Comment 3 Florian Best univentionstaff 2014-11-28 16:33:32 CET
*** Bug 36941 has been marked as a duplicate of this bug. ***
Comment 4 Florian Best univentionstaff 2014-11-28 17:48:15 CET
(In reply to Alexander Kläser from comment #2)
> Created attachment 6482 [details]
> Screenshot network settings page after applying the values has finished
> 
> If I see correctly, the progress bar needs to be adapted in order to account
> for the correct number of scripts being executed (see screenshot).
fixed

> I spotted a minor thing, in setup.log I could not spot a __NAME___ line for
> the 10interfaces script. The patch bellow will fix that. Could you please
> apply it?
fixed
Comment 5 Alexander Kläser univentionstaff 2014-11-28 20:46:55 CET
Looks fine now, also the full progress bar during the setup wizard looks good (well, it does not reach 100%, same as before).
YAML entry is fine.
Comment 6 Moritz Muehlenhoff univentionstaff 2014-12-04 12:22:59 CET
http://errata.univention.de/ucs/4.0/2.html