Bug 28635 - Image Size aus dem Profil wird nicht mehr ausgewertet
Image Size aus dem Profil wird nicht mehr ausgewertet
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Virtual machines (UVMM)
UCS 3.0
All Linux
: P5 normal (vote)
: UCS 3.1
Assigned To: Florian Best
Stefan Gohmann
: interim-3
Depends on: 26857
Blocks: 23445
  Show dependency treegraph
 
Reported: 2012-09-24 15:30 CEST by Stefan Gohmann
Modified: 2012-12-12 21:08 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 Stefan Gohmann univentionstaff 2012-09-24 15:30:06 CEST
Seit dem Update von UCS 3.0-1 auf UCS 3.0-2 wird beim Anlegen eines Windows 7 Systems über das Profil die Vorgabe für die Festplattengröße nicht mehr verwendet. 

Im Profil sind 32GB hinterlegt, diese werden auf einem UCS 3.0-1 auch vorausgewählt beim Hinzufügen einer Festplatte im Wizard.

Mit den UVMM Paketen aus UCS 3.0-2 werden dann nur 12 GB vorgeschlagen. Das müsste der Default Wert sein.

Das Problem tritt mit UCS 3.1 und UCS 3.0-2 auf. Wenn ich die UVMM Pakete downgrade, dann funktioniert es wieder:

univention-install univention-virtual-machine-manager-daemon=1.0.105-1.372.201202221231 python-univention-virtual-machine-manager=1.0.105-1.372.201202221231 univention-management-console-module-uvmm=1.0.105-1.372.201202221231


+++ This bug was initially created as a clone of Bug #27747 +++

Gibt man bei der Hauptspeichergröße für eine VM etwas Falsches ein, hat die VM
anschließend "-1024 B".
Hier sollte die Eingabe validiert werden, bevor der Benutzer eine Change hat,
die Daten abzuspeichern.
Comment 1 Stefan Gohmann univentionstaff 2012-09-24 15:30:32 CEST
Das sollten wir zur 3.1 beheben.
Comment 2 Alexander Kläser univentionstaff 2012-09-25 16:59:30 CEST
Vielleicht ein Problem im Zuge der Umstellung von Dojo? → Bug 28551
Comment 3 Stefan Gohmann univentionstaff 2012-09-25 20:13:59 CEST
(In reply to comment #2)
> Vielleicht ein Problem im Zuge der Umstellung von Dojo? → Bug 28551

Nein, das Problem konnte auf 3.0-2 nachgestellt werden, Mit UCS 3.0-1 funktionierte es noch. Es könnte an den Änderungen zu Bug #27747 liegen.
Comment 4 Philipp Hahn univentionstaff 2012-10-09 11:54:07 CEST
Das wurde durch das Update auf Dojo-1.8 in svn35510 kaputt gemacht:
  bug fixes for Dojo update; Bug #26857

--- a/branches/ucs-3.1/ucs/virtualization/univention-virtual-machine-manager-daemon/umc/js/uvmm/DriveWizard.js
+++ b/branches/ucs-3.1/ucs/virtualization/univention-virtual-machine-manager-daemon/umc/js/uvmm/DriveWizard.js
@@ -150,7 +150,7 @@ define([
 						type: 'TextBox',
 						required: true,
 						label: _('Size (default unit MB)'),
-						value: this.domain.profileData && this.domain.profileData.diskspace ? this.domain.profileData.diskspace : '12.0 GB'
+						value: lang.getObject('domain.profileData.diskspace', false, this) || '12.0 GB'
 					}, {
 						name: 'pool_exists',
 						type: 'ComboBox',
Comment 5 Philipp Hahn univentionstaff 2012-10-09 13:58:57 CEST
Debugging mit FireBug hat gezeigt, daß beim Erzeugen des DriveWizards() die übergebenen Argumente (u.a. this.domain) nicht in die neu erzeugte Instanz übergeben werden und deswegen dort this.domain=null ist.

Laut <http://dojotoolkit.org/reference-guide/1.8/dojo/_base/declare.html#using-mixin-in-a-class-constructor> müsste es in DriveWizard.js so heißen:
-		constructor: function() {
+		constructor: function(kwArgs) {
+			lang.mixin(this, kwArgs);
			lang.mixin(this, {

Laut Alex könnte es auch so funktionieren:
-		constructor: function(kwArgs) {
+		postMixinPorperties: function() {
+			this.inherited(arguments);
			lang.mixin(this, {

Ursache ist daß DriveWizard nicht länger von umc.i18n.Mixin abgeleitet wird, was bisher in <ucs-3.0/ucs/univention-management-console-frontend/umc/i18n.js:164> die Parameter hereingemixt hat:
	constructor: function(params) {
		dojo.mixin(this, params);
	},

Da das vermutlich nicht nur den UVMM betrifft, sollten hier besser _alle_ umgestellten Module nochmal angeschaut werden.
Comment 6 Philipp Hahn univentionstaff 2012-10-26 15:33:13 CEST
Für Bug #23445 habe ich jetzt erstmal in univention-virtual-machine-manager-daemon/umc/js/uvmm/DriveWizard.js:51#constructor() einen Workaround eingefügt.

svn36679, univention-virtual-machine-manager-daemon_2.0.12-1.400.201210261528
ChangeLog: ±0
Comment 7 Florian Best univentionstaff 2012-10-29 13:53:02 CET
(In reply to comment #6)
> Für Bug #23445 habe ich jetzt erstmal in
> univention-virtual-machine-manager-daemon/umc/js/uvmm/DriveWizard.js:51#constructor()
> einen Workaround eingefügt.
> 
> svn36679, univention-virtual-machine-manager-daemon_2.0.12-1.400.201210261528
> ChangeLog: ±0
In univention-virtual-machine-manager-daemon (2.0.13-1) gefixt.

Das lag übrigens nicht an der Dojo1.8 umstellung, sondern daran, dass auf eine bisher nicht existierende Variable zugegriffen wurde. (Müsste demnach auch in 3.0-2 schon nicht mehr funktioniert haben).
Comment 8 Stefan Gohmann univentionstaff 2012-11-16 12:37:37 CET
OK, das funktioniert jetzt wieder.
Comment 9 Stefan Gohmann univentionstaff 2012-12-12 21:08:08 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

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