Bug 27751 - "this.domain is null" in DriveGrid.js durch DomainPage.js
"this.domain is null" in DriveGrid.js durch DomainPage.js
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Virtual machines (UVMM)
UCS 3.0
Other Linux
: P5 normal (vote)
: UCS 3.0-2
Assigned To: Philipp Hahn
Jürgen Kahrs
: interim-2
Depends on:
Blocks: 26645
  Show dependency treegraph
 
Reported: 2012-06-27 10:18 CEST by Philipp Hahn
Modified: 2012-07-20 15:24 CEST (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): Troubleshooting, Usability
Max CVSS v3 score:
hahn: Patch_Available+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2012-06-27 10:18:28 CEST
_uvmm/DriveGrid.js:

81	    return undefined !== this.domain.domainURI;

weil in _uvmm/DomainPage.js

474	    this._driveStore.setData(this._domain.disks);
...
476	    this._driveGrid.set('domain', this._domain);

scheinbar durch den Aufruf von _driveStore.setData() bereits das driveGrid aktualisiert wird, bevor diesem per _driveGrid.set() die Domain übergeben wurde.

Es scheint zu reichen, wenn man die 3 this._*.set('domain', this._domain)-Aufrufe vor die beiden anderen Aufrufe verschiebt:

473	    this._snapshotGrid.set('domain', this._domain);
474	    this._driveGrid.set('domain', this._domain);
475	    this._interfaceGrid.set('domain', this._domain);
476	    this._interfaceStore.setData(this._domain.interfaces);
477	    this._driveStore.setData(this._domain.disks);

PS: Durch svn33783 wurde der Code in DriveGrid.js geändert, ggf. tritt das mit der aktuellsten version für UCS-3.0-2 (interim2) schon nicht mehr auf.
Comment 1 Jascha Geerds univentionstaff 2012-06-27 11:22:02 CEST
> PS: Durch svn33783 wurde der Code in DriveGrid.js geändert, ggf. tritt das mit
> der aktuellsten version für UCS-3.0-2 (interim2) schon nicht mehr auf.

Die Änderungen aus svn33783 stammen von mir und wurden für Bug #25134 durchgeführt. Meine Änderungen sollten den hier beschriebenen Bug nicht beeinflussen, dieser müsste dementsprechend immer noch vorhanden sein.
Comment 2 Philipp Hahn univentionstaff 2012-06-27 14:02:15 CEST
Im Rahmen von Bug #26645 mitbehoben

svn33829, univention-virtual-machine-manager-daemon_1.0.111-1.378.201206271323

ChangeLog: svn13657
\item A ordering bug during initialization was fixed (\ucsBug{27751}).

Zum Nachstellen: FireBug öffnen und dann beim UMC anmelden, UVMM öffnen, VM öffnen, auf den Snapshot-Reiter wechseln. Dabei sollte im Console-Log dann die Meldung (nicht) zu sehen sein.
Comment 3 Jürgen Kahrs univentionstaff 2012-06-29 09:19:37 CEST
Die ursprüngliche Fehlermeldung tritt in UVMM auf, für diesen Test brauchte ich also einen DC master, der so konfiguriert ist, dass er VMs für einen VM server verwaltet und auch eine VM, die tatsächlich konfiguriert ist. Nur dort kann man im UMC eine UVMM öffnen.

Als der DC master und seine UVMM anfangs noch auf UCS 3.0-1 waren, konnte ich das Problem tatsächlich genau so reproduzieren, wie Philip es beschrieben hat. Danach hab ich den kompletten upgrade auf UCS 3.0-2 gemacht, bis ich das von Philip beschriebene Paket auf dem DC master installiert hatte:

dpkg -l | grep univention-virtual-machine-manager-daemo
ii  univention-virtual-machine-manager-daemon       1.0.111-1.378.201206271323                        UCS Virtual Machine Manager Daemon

Nach dem upgrade habe ich mit dem browser (und Firebug) im UMC -> UVMM -> VM -> snapshot die o.g. Fehlermeldung mehr provozieren können. Das Problem tritt also in UCS 3.0-2 nicht mehr auf.
Comment 4 Stefan Gohmann univentionstaff 2012-07-20 15:24:20 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".