Bug 27792 - Progressbar im Domain-Join-Modul statt Log-Datei
Progressbar im Domain-Join-Modul statt Log-Datei
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P2 enhancement (vote)
: UCS 3.1-1
Assigned To: Florian Best
Felix Botner
:
: 19562 25559 30863 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-02 17:38 CEST by Alexander Kläser
Modified: 2013-03-25 19:57 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): Cleanup, Usability
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-07-02 17:38:14 CEST
In mehreren UMC-Modulen werden die Log-Dateien während der Ausführung eines Kommandos angezeigt:
 * Online Updater
 * Paketverwaltung
 * Domain-Join
Gut wäre es, wenn es ein Log-Widget gäbe und wenn eine serverseitige Funktionalität zum Abrufen der neuen Zeilen (bspw. in univention-management-console-module-lib oder in einer eigenen Lib) mitgebracht werden, die von allen Modulen benutzt werden können.
Comment 1 Alexander Kläser univentionstaff 2012-09-18 17:58:44 CEST
(In reply to comment #0)
> In mehreren UMC-Modulen werden die Log-Dateien während der Ausführung eines
> Kommandos angezeigt:
>  * Online Updater
>  * Paketverwaltung
>  * Domain-Join

In der Paketverwaltung wird bereits eine Progressbar benutzt und im Updater ist dies auch geplant (Bug 25558). Daher wäre es sinnvoll, im Domain-Join ebenfalls eine Progressbar statt eines generischen Log-Viewers zu verwenden.
Comment 2 Dirk Wiesenthal univentionstaff 2012-11-30 18:38:06 CET
Die Logansicht im Join-Modul hat so ihre Macken:
 * Der Button "Volle Ansicht" verschwindet nicht, wenn man bereits darauf geklickt hat (aber was soll der Knopf noch danach?)
 * Jede Sekunde springt die Ansicht wieder ganz nach unten, man kann nicht vernünftig scrollen, bis das ganze Prozedere beendet ist.

Also:
+1 für diesen Bug hier.

Vielleicht sollte man ihn mit Bug #8402 flankieren.
Comment 3 Alexander Kläser univentionstaff 2013-02-12 12:08:59 CET
(In reply to comment #2)
> Die Logansicht im Join-Modul hat so ihre Macken:
>  * Der Button "Volle Ansicht" verschwindet nicht, wenn man bereits darauf
> geklickt hat (aber was soll der Knopf noch danach?)
>  * Jede Sekunde springt die Ansicht wieder ganz nach unten, man kann nicht
> vernünftig scrollen, bis das ganze Prozedere beendet ist.
> 
> Also:
> +1 für diesen Bug hier.
> 
> Vielleicht sollte man ihn mit Bug #8402 flankieren.

Genau, gut wäre es, wenn man sich unabhängig einfach das Logfile anzeigen lassen könnte, aber der Vorgang des Joins als Progressbar angezeigt wird.
Comment 4 Alexander Kläser univentionstaff 2013-02-12 12:09:36 CET
*** Bug 25559 has been marked as a duplicate of this bug. ***
Comment 5 Stefan Gohmann univentionstaff 2013-02-19 08:27:59 CET
*** Bug 19562 has been marked as a duplicate of this bug. ***
Comment 6 Florian Best univentionstaff 2013-02-22 13:53:50 CET
yaml: 2013-02-22-univention-join.yaml
fix: univention-join 5.0.20-1.409.201302221348
Comment 7 Stefan Gohmann univentionstaff 2013-02-26 07:42:02 CET
Das Modul scheint nach dem Errata Update nicht übersetzt zu sein. Die meisten Meldungen sind englisch.
Comment 8 Florian Best univentionstaff 2013-02-27 15:51:39 CET
(In reply to comment #7)
> Das Modul scheint nach dem Errata Update nicht übersetzt zu sein. Die meisten
> Meldungen sind englisch.
Ist angepasst, zusätzlich wurde noch eine vergessene Abhängigkeit auf python-dnspython hinzugefügt.

errata: univention-join 5.0.20-1.411.201302271422
3.1-1: univention-join 5.0.27-1
Comment 9 Alexander Kläser univentionstaff 2013-03-01 17:07:47 CET
Folgende Punkte sind noch aufgefallen:

* Auf einem Master sollte die Aktion "Neu joinen" nicht verfügbar sein. (Die Backend-Aktion sollte auch diesen Fall überprüfen.)

* Auch wenn einzelne Join-Skripte ausgeführt werden, kommt die Meldung "Der Join-Vorgang war erfolgreich". Schön wäre es hier noch etwas differenzierter Feedback zu geben.

* "Bitte bestätige, die gewählten Join-Skripte auszuführen:" könnte besser heißen "Die folgenden Join-Skripte werden ausgeführt:"

* Wenn Join-Skripte ausgeführt werden und das Join-Modul ein zweites mal aufgerufen wird, ist die Liste leer. Gut wäre es, einfach die aktuelle Liste anzuzeigen. Wichtig ist es, ein erneutes, gleichzeitiges Ausführen zu verhindern. Da beide Instanzen auf den gleichen Module-Store zu greifen, kann am Ende des Vorgangs onChange() auf dem Module-Store aufgerufen werden. Dadurch werden in allen offenen Tabs die Werte aktualisiert. Hier scheint es auch noch ein Problem zu geben, nach einem onChange() ist das Grid leer:

>  require('umc/store')('script', 'join/scripts').onChange()

* "Ein Domänencontroller master sollte vom Basis Einstellungen Modul gejoined werden." → Vorschlag: "Ein DC Master sollte über das Modul Basis-Einstellungen gejoint werden." (der Link ist cool :) )

* Wenn ich Einträge aus /var/univention-join/status herausnehmen und das Join-Modul neu öffne, werden diese Einträge nicht als ausführbar markiert.
Comment 10 Florian Best univentionstaff 2013-03-04 15:41:29 CET
(In reply to comment #9)
> Folgende Punkte sind noch aufgefallen:
> 
> * Auf einem Master sollte die Aktion "Neu joinen" nicht verfügbar sein. (Die
> Backend-Aktion sollte auch diesen Fall überprüfen.)
Ja, im frontend wird der button jetzt nicht mehr angezeigt (anstatt disabled). Im backend wurde dies schon verhindert.

> * Auch wenn einzelne Join-Skripte ausgeführt werden, kommt die Meldung "Der
> Join-Vorgang war erfolgreich". Schön wäre es hier noch etwas differenzierter
> Feedback zu geben.
OK

> * "Bitte bestätige, die gewählten Join-Skripte auszuführen:" könnte besser
> heißen "Die folgenden Join-Skripte werden ausgeführt:"
OK

> * Wenn Join-Skripte ausgeführt werden und das Join-Modul ein zweites mal
> aufgerufen wird, ist die Liste leer. Gut wäre es, einfach die aktuelle Liste
> anzuzeigen. Wichtig ist es, ein erneutes, gleichzeitiges Ausführen zu
> verhindern. Da beide Instanzen auf den gleichen Module-Store zu greifen, kann
> am Ende des Vorgangs onChange() auf dem Module-Store aufgerufen werden. Dadurch
> werden in allen offenen Tabs die Werte aktualisiert. Hier scheint es auch noch
> ein Problem zu geben, nach einem onChange() ist das Grid leer:
> 
> >  require('umc/store')('script', 'join/scripts').onChange()
OK

> * "Ein Domänencontroller master sollte vom Basis Einstellungen Modul gejoined
> werden." → Vorschlag: "Ein DC Master sollte über das Modul Basis-Einstellungen
> gejoint werden." (der Link ist cool :) )
OK

> * Wenn ich Einträge aus /var/univention-join/status herausnehmen und das
> Join-Modul neu öffne, werden diese Einträge nicht als ausführbar markiert.
Ja, gefixt.
Comment 11 Alexander Kläser univentionstaff 2013-03-06 16:18:52 CET
Weitere Punkte:

* Auswahl von 2 bereits ausgeführten + 2 noch nicht ausgeführten Skripten → Ausführen → Benachrichtigung, dass 2 Skript ausgeführt werden → nach Ausführen wird das Grid nicht aktualisiert, bei Neustart des Moduls ist die Anzeige korrekt (Chromium)

* Bei Eingabe falscher Credentials erhalte ich die Fehlermeldung:

"Der system joinprozess konnte nicht fertiggestellt werden: binddn for user dsfasf not found. Weitere informationen können in der Logdatei /var/log/univention/join.log gefunden werden. Bitte Vorgang nach lösen aller Konflikte wiederholen."

Vielleicht könnte man hier noch ein paar Zeilenumbrüche einbauen, da die Fehlermeldung des Servers etwas untergeht, bspw. (mit ein paar Korrekturen):

"Der Join-Vorgang konnte nicht durchgeführt werden:

<i>binddn for user dsfasf not found</i>

Weitere _I_nformationen können in der Logdatei <i>/var/log/univention/join.log</i> gefunden werden. Bitte den Join-Vorgang nach _L_ösen aller Konflikte erneut durchführen."

* Könnte initial in der unteren Zeile der Progressbar "Authentifizierung" bei "Join skripte ausführen" angezeigt werden?

* "Join skripte ausführen..." vielleicht besser → "Ausführen der Join-Skripte" (Schreibweise mit "-" bitte beachten)

* Kann die Sortierung per default zuerst den Status (noch nicht ausgeführte Join-Skripte stehen oben) und dann den Namen berücksichtigen? Dann würde man einen sehr guten Überblick über noch nicht ausgeführte Join-Skripte haben.

* "Join-Prozess" → sollte eher "Join-Vorgang" heißen

* Hm… eigentlich müsste beim Reload des Grids eine Standby-Animation kommen

(In reply to comment #10)
> ...
> > * "Ein Domänencontroller master sollte vom Basis Einstellungen Modul gejoined
> > werden." → Vorschlag: "Ein DC Master sollte über das Modul Basis-Einstellungen
> > gejoint werden." (der Link ist cool :) )
> OK

→ noch nicht ganz OK, bitte Groß-/Kleinschreibung und andere Typos berücksichtigen.

* "Bitte Benutzernamen und Passwort eines Domänen-Administrators eingeben,..."
→ vielleicht besser:
Bitte Zugangsdaten eines Benutzerkontos mit Administratorrechten eingeben, ..."

* "Um das System zu joinen bitte die benötigten Informationen ausfüllen und den 'System joinen' Button klicken."
→ Vielleicht auch noch mit den Informationen ähnlich zum übergeordneten Punkt erweitern.

* "Führe join skript..." → "Ausführung des Join-Skripts ..."

* Wenn ich manuell Join-Skripte starten, kann ich derzeit deren Ausführreihenfolge bestimmen (Skripte mit hoher Nummer zuerst), indem ich die Sortierung des Grids ändere. Hier sollte im Backend die Liste der zu ausführenden Skripte immer sortiert werden.
Comment 12 Florian Best univentionstaff 2013-03-07 16:04:46 CET
(In reply to comment #11)
> Weitere Punkte:
> 
> * Auswahl von 2 bereits ausgeführten + 2 noch nicht ausgeführten Skripten →
> Ausführen → Benachrichtigung, dass 2 Skript ausgeführt werden → nach Ausführen
> wird das Grid nicht aktualisiert, bei Neustart des Moduls ist die Anzeige
> korrekt (Chromium)
OK

> * Bei Eingabe falscher Credentials erhalte ich die Fehlermeldung:
> 
> "Der system joinprozess konnte nicht fertiggestellt werden: binddn for user
> dsfasf not found. Weitere informationen können in der Logdatei
> /var/log/univention/join.log gefunden werden. Bitte Vorgang nach lösen aller
> Konflikte wiederholen."
> 
> Vielleicht könnte man hier noch ein paar Zeilenumbrüche einbauen, da die
> Fehlermeldung des Servers etwas untergeht, bspw. (mit ein paar Korrekturen):
OK

> * Könnte initial in der unteren Zeile der Progressbar "Authentifizierung" bei
> "Join skripte ausführen" angezeigt werden?
OK

> * "Join skripte ausführen..." vielleicht besser → "Ausführen der Join-Skripte"
> (Schreibweise mit "-" bitte beachten)
OK

> * Kann die Sortierung per default zuerst den Status (noch nicht ausgeführte
> Join-Skripte stehen oben) und dann den Namen berücksichtigen? Dann würde man
> einen sehr guten Überblick über noch nicht ausgeführte Join-Skripte haben.
OK

> * "Join-Prozess" → sollte eher "Join-Vorgang" heißen
OK

> * Hm… eigentlich müsste beim Reload des Grids eine Standby-Animation kommen
OK, das lag daran, dass ich die filter funktion vom dojo grid anstatt dem umc grid benutzt hatte.

> (In reply to comment #10)
> > ...
> > > * "Ein Domänencontroller master sollte vom Basis Einstellungen Modul gejoined
> > > werden." → Vorschlag: "Ein DC Master sollte über das Modul Basis-Einstellungen
> > > gejoint werden." (der Link ist cool :) )
> > OK
> 
> → noch nicht ganz OK, bitte Groß-/Kleinschreibung und andere Typos
> berücksichtigen.
OK

> * "Bitte Benutzernamen und Passwort eines Domänen-Administrators eingeben,..."
> → vielleicht besser:
> Bitte Zugangsdaten eines Benutzerkontos mit Administratorrechten eingeben, ..."
OK
> * "Um das System zu joinen bitte die benötigten Informationen ausfüllen und den
> 'System joinen' Button klicken."
> → Vielleicht auch noch mit den Informationen ähnlich zum übergeordneten Punkt
> erweitern.
OK

> * "Führe join skript..." → "Ausführung des Join-Skripts ..."
OK

> * Wenn ich manuell Join-Skripte starten, kann ich derzeit deren
> Ausführreihenfolge bestimmen (Skripte mit hoher Nummer zuerst), indem ich die
> Sortierung des Grids ändere. Hier sollte im Backend die Liste der zu
> ausführenden Skripte immer sortiert werden.
Ja, das wird nun gemacht.


univention-join (5.0.35-1)
errata: 5.0.20-1.423.201303071553
yaml updated
Comment 13 Alexander Kläser univentionstaff 2013-03-19 16:08:09 CET
Wäre es möglich, die Sortierung zuerst nach Status und dann nach Skriptname durchzuführen? Aktuell hat man keine Chance ein Skript zu finden.
Comment 14 Alexander Kläser univentionstaff 2013-03-19 16:13:01 CET
Please remove the YAML file and add comments to the 3.1-1 changelog.
Comment 15 Florian Best univentionstaff 2013-03-19 16:33:48 CET
(In reply to comment #14)
> Please remove the YAML file and add comments to the 3.1-1 changelog.
removed yaml, patches, rebuild errata packages wihtout patches, addded 3.1-1 changelog.

(In reply to comment #13)
> Wäre es möglich, die Sortierung zuerst nach Status und dann nach Skriptname
> durchzuführen? Aktuell hat man keine Chance ein Skript zu finden.
Ja, das lag an einem typo (ein ',' zuviel).
Comment 16 Alexander Kläser univentionstaff 2013-03-19 17:31:09 CET
(In reply to comment #15)
> (In reply to comment #13)
> > Wäre es möglich, die Sortierung zuerst nach Status und dann nach Skriptname
> > durchzuführen? Aktuell hat man keine Chance ein Skript zu finden.
> Ja, das lag an einem typo (ein ',' zuviel).

Funkioniert immer noch nicht.

Vorschlag: Vom Backend können als Statuseigenschaft Status + Name zurückgegeben werden (bspw. status="0:35univention-management-console-module-appcenter"). Nativ kann dann sortiert werden und für die Anzeige wird die Eigenschaft "configured" ausgewertet und ein entsprechender String zurückgegeben werden.
Comment 17 Alexander Kläser univentionstaff 2013-03-19 17:45:49 CET
* Auf einem Backup, beim initialen Join steht in Überschrift + TitlePane:

"der Domäne beitreten" → besser "Initialer Domänenbeitritt" + "Zugangsdaten"

* In der Progressbar steht beim initialen Join auf einem Backup über der Bar "Domänebeitritt" → Typo

* Bei der Ausführung von Join-Skripten auf einem Slave steht über der Progressbar die ganze Zeit "Authentication"

Sonst sieht alles sehr gut aus :) .
Comment 18 Florian Best univentionstaff 2013-03-20 09:47:58 CET
(In reply to comment #17)
> * Auf einem Backup, beim initialen Join steht in Überschrift + TitlePane:
> 
> "der Domäne beitreten" → besser "Initialer Domänenbeitritt" + "Zugangsdaten"
OK

> * In der Progressbar steht beim initialen Join auf einem Backup über der Bar
> "Domänebeitritt" → Typo
OK

> * Bei der Ausführung von Join-Skripten auf einem Slave steht über der
> Progressbar die ganze Zeit "Authentication"
OK
Comment 19 Felix Botner univentionstaff 2013-03-21 15:07:52 CET
Es gibt einen Fehler, wenn man "Rechnername des Domänencontroller Master" leer läßt.

Beim Join mit einem falschen Passwort (oder allg. bei einem Fehler) verstellt der "UMC Neuladen" Dialog die Fehlermeldung, das ist etwas ungünstig

Wenn ich Benutzername und Passwort leer lasse, sagt er "Join erfolgreich".
Comment 20 Florian Best univentionstaff 2013-03-22 08:58:49 CET
(In reply to comment #19)
> Es gibt einen Fehler, wenn man "Rechnername des Domänencontroller Master" leer
> läßt.
> 
> Beim Join mit einem falschen Passwort (oder allg. bei einem Fehler) verstellt
> der "UMC Neuladen" Dialog die Fehlermeldung, das ist etwas ungünstig
> 
> Wenn ich Benutzername und Passwort leer lasse, sagt er "Join erfolgreich".

Ok, das sollte nun nicht mehr möglich sein. Bitte auch einmal testen, ob auf einem master die join skripte ausgeführt werden können.
Comment 21 Felix Botner univentionstaff 2013-03-22 09:17:22 CET
Sieht soweit gut aus.

TODO: Appliance
Comment 22 Florian Best univentionstaff 2013-03-22 09:19:31 CET
*** Bug 30863 has been marked as a duplicate of this bug. ***
Comment 23 Felix Botner univentionstaff 2013-03-22 09:42:01 CET
OK - Master Skripte erneut ausführen
OK - Non-Master erneut joinen
OK - Progressbar
OK - Join Skripte ausführen

OK - Changelog
Comment 24 Stefan Gohmann univentionstaff 2013-03-25 19:57:23 CET
UCS 3.1-1 has been released: 
 http://download.univention.de/doc/release-notes-3.1-1_en.pdf
 http://download.univention.de/doc/release-notes-3.1-1.pdf

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