Bug 23931 - Protokollerweiterung
Protokollerweiterung
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Virtualization - UVMM
UCS 3.0
Other Linux
: P5 normal (vote)
: UCS 3.0 - RC
Assigned To: Andreas Büsching
Philipp Hahn
: interim-1
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-10-05 09:44 CEST by Andreas Büsching
Modified: 2011-12-13 15:51 CET (History)
3 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 Andreas Büsching univentionstaff 2011-10-05 09:44:35 CEST
Für das neue UVMM UMC-Modul wäre es sinnvoll, wenn das Protokoll es erlaubt kleinere informationsbrocken abzufragen. Zum Beispiel:

- Liste der Gruppen
- Liste der Server einer Gruppe
- Details zu einer Instanz
Comment 1 Philipp Hahn univentionstaff 2011-10-05 11:37:30 CEST
(In reply to comment #0)
> Für das neue UVMM UMC-Modul wäre es sinnvoll, wenn das Protokoll es erlaubt
> kleinere informationsbrocken abzufragen. Zum Beispiel:

> - Liste der Gruppen
Das gibt es schon "uvmm groups" → univention.uvmm.command.GROUP_LIST

> - Liste der Server einer Gruppe
Das gibt es schon "uvmm nodes $group" → univention.uvmm.command.NODE_LIST

> - Details zu einer Instanz
Hier sollte NODE_QUERY aufgespalten werden in:
1. nur Liste der Domain-UUIDs und -Namen zurückgliefern, ggf. auch noch gesamt-CPU.Verbrauch, gesamt-RAM-Auslastung, etc.
2. die Details zu einer einzelnen VM
Bisher liefert NODE_QUERY immer detaillierte Informationen zu allen VMs eines Servers.
Comment 2 Andreas Büsching univentionstaff 2011-10-07 12:22:12 CEST
(In reply to comment #1)
> > - Details zu einer Instanz
> Hier sollte NODE_QUERY aufgespalten werden in:
> 1. nur Liste der Domain-UUIDs und -Namen zurückgliefern, ggf. auch noch
> gesamt-CPU.Verbrauch, gesamt-RAM-Auslastung, etc.

Hier wird jetzt eine Liste mit ( UUID, Name ) Paaren zurückgegeben
Comment 3 Andreas Büsching univentionstaff 2011-10-10 08:42:50 CEST
(In reply to comment #1)
> 2. die Details zu einer einzelnen VM
> Bisher liefert NODE_QUERY immer detaillierte Informationen zu allen VMs eines
> Servers.

ist jetzt ebenfalls umgesetzt
Comment 4 Philipp Hahn univentionstaff 2011-11-09 11:14:19 CET
(In reply to comment #0)
> - Liste der Gruppen

OK: uvmm groups

> - Liste der Server einer Gruppe

OK: uvmm nodes $group
OK: uvmm nodes $group qemu\*
OK: uvmm nodes $group \*/system

> - Details zu einer Instanz

OK: uvmm domains $uri
OK: uvmm domains $uri m\*
OK: uvmm domains $uri \*t

OK: uvmm domain_info $uri $uuid

FAIL:
Hier ist eine inkompatible Änderung am UVMM-Protokoll erfolgt:
"uvmm nodes $group" liefert jetzt nicht mehr nur eine Liste von URIs zurück, sondern eine Liste von Data_Node-Objekten. Damit funktioniert ein alter UVMMd nicht mehr mit einem neuen UMC-Module und umgekehrt.
Auch werden jetzt statt Identifiern (Zahlen) direkt die XML-Zeichenketten verwendet.

Deshalb sollte die Protokoll-Nummer erhöht werden, damit es nicht erneut zu schwierig zu debuggenden Problemen wie heute auf Xen5 kommt.

$ grep -Hn ^VERSION $PWD/protocol.py 
branches/ucs-3.0/ucs/virtualization/univention-virtual-machine-manager-daemon/src/univention/uvmm/protocol.py:45:VERSION = (1, 2)

Da die Änderungen nicht abwärtskompatibel ist, ist hier (2,0) angebracht.

FAIL: kein ChangeLog
halte ich wegen der Protokoll-Änderung für notwendig.
Comment 5 Andreas Büsching univentionstaff 2011-11-25 21:28:26 CET
(In reply to comment #4)
> $ grep -Hn ^VERSION $PWD/protocol.py 
> branches/ucs-3.0/ucs/virtualization/univention-virtual-machine-manager-daemon/src/univention/uvmm/protocol.py:45:VERSION
> = (1, 2)

fixed.
 
> Da die Änderungen nicht abwärtskompatibel ist, ist hier (2,0) angebracht.
> 
> FAIL: kein ChangeLog
> halte ich wegen der Protokoll-Änderung für notwendig.

fixed.
Comment 6 Philipp Hahn univentionstaff 2011-11-25 21:31:45 CET
(In reply to comment #5)
> (In reply to comment #4)
> > $ grep -Hn ^VERSION $PWD/protocol.py 
> > branches/ucs-3.0/ucs/virtualization/univention-virtual-machine-manager-daemon/src/univention/uvmm/protocol.py:45:VERSION = (1, 2)
> 
> fixed.

OK: svn29606

> > FAIL: kein ChangeLog
> > halte ich wegen der Protokoll-Änderung für notwendig.
> 
> fixed.

OK: svn11153 (auch wenn da ein Punkt am Ende des Eintrags fehlt)
Comment 7 Sönke Schwardt-Krummrich univentionstaff 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"