Comment 1 Andreas Büsching 2011-06-28 08:41:40 CEST

*** Bug 22897 has been marked as a duplicate of this bug. ***

Comment 2 Jascha Geerds 2011-07-21 14:15:27 CEST

Das ist erledigt.

Comment 3 Alexander Kläser 2011-07-21 17:59:07 CEST

QA: Das Modul sieht gut aus! Die Funktionalitäten klappen soweit: Suche, Sortierung, Beenden von Prozessen. Folgende Punkte sind aufgefallen:

* Ich glaube, dass das neue Paket noch gebaut werden muss.

* Nicht alle Prozesse haben einen entsprechenden Kommandozeilebefehl. Mit 'ps afx' werden diese Befehle als '[<name>]' ausgegeben. Das wäre sicherlich nett, diese Verhalten auch für das Modul zu übernehmen. Dieses Feld könnte dann auch beim Suchen berücksichtigt werden.

* Wird ein Prozess gekillt, der bereits gekillt wurde, dann kommt eine Fehlermeldung im Frontend. In der Logdatei steht folgender Traceback:

Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py", line 91, in execute
    func( request )
  File "/usr/lib/pymodules/python2.6/univention/management/console/modules/top/__init__.py", line 88, in kill
    MODULE.error(str(error))
NameError: global name 'MODULE' is not defined

* Die Nachricht 'Processes killed successfully' ist IMHO etwas irreführend, bspw. wenn ein Prozess mit SIGTERM gekillt wird, sich aber nicht beendet, wie z.B. das folgende Skript:

#!/bin/bash
trap "echo NOPE" SIGINT SIGTERM
echo "pid is $$"
while : ; do 
  sleep 60;
done

Vielleicht sollte man lieber die Nachricht ausgeben, dass das entsprechende Signal an die ausgewählten Prozesse gesendet wurde.

Comment 4 Jascha Geerds 2011-08-15 17:11:00 CEST

Alle angemerkten Probleme wurden behoben.


Fixed!

Comment 5 Alexander Kläser 2011-08-16 17:23:58 CEST

(In reply to comment #3)
> QA: Das Modul sieht gut aus! Die Funktionalitäten klappen soweit: Suche,
> Sortierung, Beenden von Prozessen. Folgende Punkte sind aufgefallen:
> 
> * Ich glaube, dass das neue Paket noch gebaut werden muss.

→ OK

> * Nicht alle Prozesse haben einen entsprechenden Kommandozeilebefehl. Mit 'ps
> afx' werden diese Befehle als '[<name>]' ausgegeben. Das wäre sicherlich nett,
> diese Verhalten auch für das Modul zu übernehmen. Dieses Feld könnte dann auch
> beim Suchen berücksichtigt werden.

→ OK, für alle werden jetzt Namen angezeigt.

> * Die Nachricht 'Processes killed successfully' ist IMHO etwas irreführend,
> bspw. wenn ein Prozess mit SIGTERM gekillt wird, sich aber nicht beendet, wie
> z.B. das folgende Skript:
> 
> #!/bin/bash
> trap "echo NOPE" SIGINT SIGTERM
> echo "pid is $$"
> while : ; do 
>   sleep 60;
> done
> 
> Vielleicht sollte man lieber die Nachricht ausgeben, dass das entsprechende
> Signal an die ausgewählten Prozesse gesendet wurde.

→ OK, das ist super so.

> * Wird ein Prozess gekillt, der bereits gekillt wurde, dann kommt eine
> Fehlermeldung im Frontend. In der Logdatei steht folgender Traceback:
> 
> Traceback (most recent call last):
>   File
> "/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py",
> line 91, in execute
>     func( request )
>   File
> "/usr/lib/pymodules/python2.6/univention/management/console/modules/top/__init__.py",
> line 88, in kill
>     MODULE.error(str(error))
> NameError: global name 'MODULE' is not defined

→ OK, Den Traceback gibt es nicht mehr.

Super wäre es noch, wenn eine Meldung wie "Der Prozess konnte nicht gefunden werden" im Fehlerfall vom Server als "message" zurückgegeben wird.
→ REOPEN

Comment 6 Alexander Kläser 2011-09-19 13:20:19 CEST

Noch ein paar Punkte aus unserer internen Feedbackrunde:
* Bevor Prozesse beendet werden muss es zwingend eine Sicherheitsabfrage geben
* Das Suchwidget könnte mit den Buttons in eine Zeile zusammengefasst werden

Comment 7 Jascha Geerds 2011-09-27 11:56:39 CEST

Alle hier genannten Punkt wurden behoben.

Comment 8 Alexander Kläser 2011-09-28 14:35:59 CEST

QA: OK → VERIFIED

Comment 9 Lukas Walter 2011-10-07 14:54:09 CEST

Zur Zeit werden im Prozessübersichts-Modul der UMC fehlerhafte Werte für die CPU-Last angezeigt. Meistens handelt es sich bei dem für einen Prozess angezeigten Wert um 0%, manchmal aber auch um einen Wert weit über 100%.

Zu dem ist es momentan nicht möglich die Prozesse nach 'Virtual(MB)' oder 'Resident(MB)' zu sortieren. In beiden Fällen wird die Meldung "Sorry, an error occurred" an Stelle einer Prozessliste ausgegeben.

Comment 10 Alexander Kläser 2011-10-07 14:58:10 CEST

Die Sortierung nach Speicher (in Prozent) funktioniert, ist allerdings derzeit noch lexikografisch und nicht numerisch.

Comment 11 Alexander Kläser 2011-10-12 15:15:16 CEST

(In reply to comment #9)
> Zur Zeit werden im Prozessübersichts-Modul der UMC fehlerhafte Werte für die
> CPU-Last angezeigt. Meistens handelt es sich bei dem für einen Prozess
> angezeigten Wert um 0%, manchmal aber auch um einen Wert weit über 100%.
> 
> Zu dem ist es momentan nicht möglich die Prozesse nach 'Virtual(MB)' oder
> 'Resident(MB)' zu sortieren. In beiden Fällen wird die Meldung "Sorry, an error
> occurred" an Stelle einer Prozessliste ausgegeben.

(In reply to comment #10)
> Die Sortierung nach Speicher (in Prozent) funktioniert, ist allerdings derzeit
> noch lexikografisch und nicht numerisch.

Beide Punkte wurden ausgelagert nach Bug 24037.

→ VERIFIED

Comment 12 Sönke Schwardt-Krummrich 2011-12-13 15:51:13 CET

UCS 3.0-0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer
neueren Version von UCS erneut auftreten, so sollte dieser Bug dupliziert
werden: "Clone This Bug"