Bug 26235 - Neustart von UMC-Webservern vereinheitlichen
Neustart von UMC-Webservern vereinheitlichen
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P2 enhancement (vote)
: UCS 3.0-2
Assigned To: Alexander Kläser
Lukas Walter
: interim-2
: 25293 26306 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-23 17:25 CET by Alexander Kläser
Modified: 2012-07-24 14:58 CEST (History)
8 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:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-02-23 17:25:51 CET
Derzeit wird der Neustart von UMC-Webserver, UMC-Server und Apache bei der Installation von Softwarekomponenten im System-Setup sowie im Updater-Modul verhindert. Die folgenden Punkte bedürfen einer Lösung:

(a) Der Neustart der Dienste wird derzeit nicht in dem Packages-Modul berücksichtigt.

(b) Der Neustart der Dienste wird zwar verhindert, dadurch ist es aber leider auch nicht möglich, eine Softwarekomponente zu installieren, für die ein Neustart der Dienste notwendig ist. Optimal wäre eine Benachrichtigung des Benutzers mit Hinweis auf die Notwendigkeit eines Neustarts der Dienste. Dieser Neustart könnte vom Benutzer durchgeführt oder abgelehnt werden. Dazu müsste beobachtet werden (evtl. mit inotify?) welche Dienste durch die Softwarekomponenten mit welchen Parametern (restart/reload) aufgerufen wurden, so dass eine entsprechende Entscheidung getroffen werden kann, ob ein Neustart notwendig ist.
Comment 1 Alexander Kläser univentionstaff 2012-03-01 13:07:37 CET
Siehe auch Bug 26306.
Comment 2 Alexander Kläser univentionstaff 2012-03-30 12:25:28 CEST
*** Bug 25930 has been marked as a duplicate of this bug. ***
Comment 3 Alexander Kläser univentionstaff 2012-03-30 12:26:52 CEST
(In reply to comment #2)
> *** Bug 25930 has been marked as a duplicate of this bug. ***

Das Fehlverhalten aus Bug 25930 bitte auch berücksichtigen.
Comment 4 Alexander Kläser univentionstaff 2012-06-25 12:54:27 CEST
*** Bug 25293 has been marked as a duplicate of this bug. ***
Comment 5 Alexander Kläser univentionstaff 2012-06-25 12:55:50 CEST
(c) Bug 25293 beschreibt ein ähnliches Problem, das in Zuge dieses Bugs ebenso berücksichtigt werden sollte: 

> Nach dem joinen eines Slaves aus dem UMC-Join Modul funktioniert der UMC-Login
> nicht mehr. Nach einem univention-management-console{,web-}-server funktioniert
> alles wieder wie gewohnt.
Comment 6 Alexander Kläser univentionstaff 2012-06-28 15:55:29 CEST
*** Bug 26306 has been marked as a duplicate of this bug. ***
Comment 7 Alexander Kläser univentionstaff 2012-06-28 18:09:10 CEST
Ein entsprechendes lib-UMC-Module wurde erstellt, als Lib für serverseitige Methoden. Die folgenden Pakete wurden bereits angepasst:


univention-management-console-module-lib (0.1.0-1) unstable; urgency=low
.
  * Initial release of module with library methods; Bug #26235

univention-system-setup (5.0.177-1) unstable; urgency=low
.
  * added dialog to restart UMC server components after software installation
  * make sure that UMC ACLs for "lib/server/*" for root and __systemsetup__
    are granted
  Bug #26235

univention-management-console (4.0.266-2) unstable; urgency=low
.
  * added UMC ACLs for command lib/* for root
  * added univention-management-console-module-lib to default dependencies
  Bug #26235
Comment 8 Alexander Kläser univentionstaff 2012-06-29 14:24:08 CEST
Die Pakete wurden soweit schon gebaut, alle Module wurden noch nicht angepasst. Folgende Änderungen wurden noch vorgenommen:


 univention-system-setup (5.0.178-1) unstable; urgency=low
 .
   * grant acces for "lib/server/*" with UMC operation set setup-all; Bug #26235

 univention-management-console-module-packages (1.0.2-2.2-1) unstable; urgency=low
 .
   * disable restart of UMC server components during installation and added
     dialog to confirm a restart of UMC server components
   * adjusted package dependencies and join script
   Bug #26235
Comment 9 Alexander Kläser univentionstaff 2012-07-02 16:17:44 CEST
Das Join-Modul ist jetzt ebenfalls angepasst, dazu musste die JavaScript-seitige Logik etwas angepasst werden. So wird die Log-Ausgabe jetzt nicht mehr paralell zum Join-Formular oder zur Grid-Ansicht gezeigt. Entweder wird die Log-Datei, das Grid oder das Join-Formular (auf einem ungejointen System) angezeigt.

Für die QA sollten die folgende, allgemeinen Szenarien für das Join-Modul noch einmal getestet werden:
* Test auf Master- und einem Nicht-Master-System
* Kompletter System-Join
* Ausführen einzelner Skripte
* Ausführen eines Vorganges, Schließen des Moduls und Wiederöffnen (ggf. auch ausloggen)
* Die UMC-Server-Komponenten (UMC Web-Server, UMC-Server, Apache) sollten während der Ausführung eines Join-Vorgangs (kompletter Join oder auch die Ausführung einzelner Skripte) nicht wiederstartbar sein
* Nach der erfolgreichen Ausführung erscheint ein Abfrage-Dialog, ob die Server-Komponenten neu gestartet werden sollen
* Bitte auch überprüfen, dass die Polling-Anfragen in der Log-Datei-Ansicht bei Schließen des Moduls ebenfalls beendet werden
* Bitte auch das korrekte Verhalten für falsche Eingaben als Credentials prüfen


 univention-join (4.0.37-1) unstable; urgency=low
 .
   * refactored UMC module to either show the join log, the grid or the form
     for initial join, no parallel view allowed anymore
   * added ask for restarting UMC server components after the join process
   * adapted UMC operation set and package dependencies to allow access to UMCP
     command lib/server/restart
   Bug #26235
Comment 10 Jascha Geerds univentionstaff 2012-07-03 13:17:38 CEST
Ich habe stellenweise schon mal mit der QA begonnen. Die zuvor mit dir besprochenen Punkte funktionieren nun:

* Der Text in dem Dialog ist lokalisiert
* Syntaxfehler in dem 'univention-join'-Modul wurde behoben
Comment 11 Alexander Kläser univentionstaff 2012-07-03 13:18:59 CEST
Die Anpassungen am Updater sind soweit auch fertig.

Für die QA: Bitte generell überprüfen, dass für in den UMC-Operation-Sets der bearbeiteten Module auch lib/server/* freigegeben wird. Bitte auch überprüfen, ob nach einem Update für root lib/* freigegebn ist (umc-acls -u root show).


 univention-updater (7.0.170-1) unstable; urgency=low
 .
   * added dialog for restarting UMC server components; Bug #26235
Comment 12 Alexander Kläser univentionstaff 2012-07-03 14:48:27 CEST
Changelog-Eintrag wurde noch hinzugefügt.
Comment 13 Alexander Kläser univentionstaff 2012-07-03 17:40:37 CEST
Fehler bzgl. des Pollings und der Fehlerbehandlung bei der Überprüfung der Credentials wurden ausgebessert.


 univention-join (4.0.41-1) unstable; urgency=low
 .
   * fixed polling issues
   * fixed backend error handling with for credentials check
   Bug #26235
Comment 14 Jascha Geerds univentionstaff 2012-07-04 11:50:01 CEST
Ich bin soweit mit der QA durch. Zwei Punkte sind jedoch noch aufgefallen:

* Der Dialog in dem gefragt wird, ob der UMC(-Web)-Server neugetartet werden soll oder nicht, taucht nicht auf, wenn das jeweilige Modul über die Tableiste geschlossen wird. Ich denke es ist gar nicht so unwahrscheinlich, dass viele Benutzer das Modul direkt in der Tableiste beenden, nachdem z.B. neue Software installiert wurde. Das sollte vielleicht noch abgefangen werden, sodass auch auf diesem Wege der Dialog erscheint.

* Im Paketverwaltungsmodul ist der o.g. Dialog noch nicht lokalisiert. Er wird immer in Englisch angezeigt.
Comment 15 Alexander Kläser univentionstaff 2012-07-04 17:08:43 CEST
(In reply to comment #14)
> Ich bin soweit mit der QA durch. Zwei Punkte sind jedoch noch aufgefallen:
> 
> * Der Dialog in dem gefragt wird, ob der UMC(-Web)-Server neugetartet werden
> soll oder nicht, taucht nicht auf, wenn das jeweilige Modul über die Tableiste
> geschlossen wird. Ich denke es ist gar nicht so unwahrscheinlich, dass viele
> Benutzer das Modul direkt in der Tableiste beenden, nachdem z.B. neue Software
> installiert wurde. Das sollte vielleicht noch abgefangen werden, sodass auch
> auf diesem Wege der Dialog erscheint.

Das ist ein generisches Problem, welches an Bug 27827 ausgelagert wurde.

> * Im Paketverwaltungsmodul ist der o.g. Dialog noch nicht lokalisiert. Er wird
> immer in Englisch angezeigt.

Das wurde angepasst in:

 univention-management-console-module-packages (1.0.5-1) unstable; urgency=low
 .
   * added missing translations
   * fixed changelog version numbers
   Bug #26235
Comment 16 Alexander Kläser univentionstaff 2012-07-04 18:25:26 CEST
Noch ein Fehler ist aufgefallen. Nach dem erfolgreichen Einspielen von Softwarepaketen und bei Abbruch des UMC-Neustarts wurde noch die Progressbar angezeigt. Dies wurde behoben.


 univention-system-setup (5.0.182-1) unstable; urgency=low
 .
   * perform a reload when asking to restart the server
   * make sure that the FQDN is not set to "." after role change
   Bug #26235
Comment 17 Lukas Walter univentionstaff 2012-07-05 14:36:41 CEST
Deutsche Lokalisierung für den restart confirm Dialog im Paketverwaltungsmodul ist jetzt vorhanden.

Bei einem Abbrechen des Neustarts im system-setup wird die Progressbar nicht angezeigt.


Was aufgefallen ist:
=====================
Der Text ("For the changes to take effect, it is recommended to perform a restart of the UMC server components. This will take approximately 10 sec.") des Dialogs im Online update Modul weicht durch die zusätzliche Zeitangabe von den Texten in den anderen Modulen ab.

Er sollte denen in den anderen Modulen mehr ähneln, etwa:
"For the changes to take effect, it is recommended to perform a restart of the UMC server components."
Comment 18 Alexander Kläser univentionstaff 2012-07-05 17:09:49 CEST
(In reply to comment #17)
> ...
> Was aufgefallen ist:
> =====================
> Der Text ("For the changes to take effect, it is recommended to perform a
> restart of the UMC server components. This will take approximately 10 sec.")
> des Dialogs im Online update Modul weicht durch die zusätzliche Zeitangabe von
> den Texten in den anderen Modulen ab.
> 
> Er sollte denen in den anderen Modulen mehr ähneln, etwa:
> "For the changes to take effect, it is recommended to perform a restart of the
> UMC server components."

Das wurde entsprechend behoben, der Hinweis bzgl. der 10 Sekunden wird nun in der eigentlichen Bibliotheksmethode angegeben.


 univention-updater (7.0.171-1) unstable; urgency=low
 .
   * modified dialog message for restarting server components; Bug #26235

 univention-management-console-module-lib (0.1.2-1) unstable; urgency=low
 .
   * adjusted dialog message; Bug #26235
Comment 19 Lukas Walter univentionstaff 2012-07-05 17:55:43 CEST
Änderungen sehen gut aus.

Verified.
Comment 20 Felix Botner univentionstaff 2012-07-13 09:47:45 CEST
Ich bekomme die Meldung

"Um das ausgeführte pdate abzuschließen, ist es erforderlich, das System neu zu starten."
Comment 21 Dirk Wiesenthal univentionstaff 2012-07-13 13:03:53 CEST
Was spricht eigentlich dagegen, statt
"In order to complete the update, it is strongly recommended to reboot the system."
das bereits andernorts genutzte (und lokalisierte)
"In order to complete the recently executed action, it is required to reboot the system"
zu nutzen?
Comment 22 Alexander Kläser univentionstaff 2012-07-13 13:22:55 CEST
(In reply to comment #20)
> Ich bekomme die Meldung
> 
> "Um das ausgeführte pdate abzuschließen, ist es erforderlich, das System neu zu
> starten."

(In reply to comment #21)
> Was spricht eigentlich dagegen, statt
> "In order to complete the update, it is strongly recommended to reboot the
> system."
> das bereits andernorts genutzte (und lokalisierte)
> "In order to complete the recently executed action, it is required to reboot
> the system"
> zu nutzen?

Variante (D) (D=Dirk) wurde umgesetzt :) .


 univention-updater (7.0.184-1) unstable; urgency=low
 .
   * fixed typos; Bug #26235
Comment 23 Lukas Walter univentionstaff 2012-07-13 15:29:44 CEST
(In reply to comment #22)
> (In reply to comment #20)
> > Ich bekomme die Meldung
> > 
> > "Um das ausgeführte pdate abzuschließen, ist es erforderlich, das System neu zu
> > starten."
> 
> (In reply to comment #21)
> > Was spricht eigentlich dagegen, statt
> > "In order to complete the update, it is strongly recommended to reboot the
> > system."
> > das bereits andernorts genutzte (und lokalisierte)
> > "In order to complete the recently executed action, it is required to reboot
> > the system"
> > zu nutzen?
> 
> Variante (D) (D=Dirk) wurde umgesetzt :) .
> 
> 
>  univention-updater (7.0.184-1) unstable; urgency=low
>  .
>    * fixed typos; Bug #26235

Lokalisierung sieht jetzt gut aus.

Verified.
Comment 24 Stefan Gohmann univentionstaff 2012-07-20 15:24:57 CEST
UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".
Comment 25 Florian Best univentionstaff 2012-07-23 10:28:46 CEST
*** Bug 11925 has been marked as a duplicate of this bug. ***
Comment 26 Florian Best univentionstaff 2012-07-24 14:58:44 CEST
*** Bug 20180 has been marked as a duplicate of this bug. ***