Update VM SCSI controller type after OS selection in Wizard
This was done previously without any kind of user visible feedback, so users would not know that they had a Virtio SCSI controller as default controller for Linux VMs. Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
parent
2414ef153a
commit
7b2f6ac8af
@ -1,7 +1,6 @@
|
||||
Ext.define('PVE.qemu.CreateWizard', {
|
||||
extend: 'PVE.window.Wizard',
|
||||
alias: 'widget.pveQemuCreateWizard',
|
||||
qemuScsiController: undefined,
|
||||
|
||||
controller: {
|
||||
|
||||
@ -167,10 +166,6 @@ Ext.define('PVE.qemu.CreateWizard', {
|
||||
var nodename = kv.nodename;
|
||||
delete kv.nodename;
|
||||
|
||||
if (me.qemuScsiController && !kv.scsihw) {
|
||||
kv.scsihw = me.qemuScsiController;
|
||||
}
|
||||
|
||||
PVE.Utils.API2Request({
|
||||
url: '/nodes/' + nodename + '/qemu',
|
||||
waitMsgTarget: me,
|
||||
|
@ -66,7 +66,7 @@ Ext.define('PVE.qemu.OSDefaults', {
|
||||
pveOS: 'w2k',
|
||||
parent : 'generic',
|
||||
networkCard: 'rtl8139',
|
||||
scsihw: 'lsi'
|
||||
scsihw: ''
|
||||
});
|
||||
// https://pve.proxmox.com/wiki/Windows_XP_Guest_Notes
|
||||
addOS({
|
||||
|
@ -27,7 +27,7 @@ Ext.define('PVE.qemu.OSTypeInputPanel', {
|
||||
|
||||
me.setWidget('pveBusSelector', targetValues.busType);
|
||||
me.setWidget('pveNetworkCardSelector', targetValues.networkCard);
|
||||
me.setWizardHiddenValue('qemuScsiController', targetValues.scsihw);
|
||||
me.setWidget('field[name=scsihw]', targetValues.scsihw);
|
||||
},
|
||||
setWidget: function(widget, newValue) {
|
||||
// changing a widget is safe only if ComponentQuery.query returns us
|
||||
@ -38,14 +38,6 @@ Ext.define('PVE.qemu.OSTypeInputPanel', {
|
||||
} else {
|
||||
throw 'non unique widget :' + widget + ' in Wizard';
|
||||
}
|
||||
},
|
||||
setWizardHiddenValue: function(property, newValue) {
|
||||
var wizards = Ext.ComponentQuery.query('pveQemuCreateWizard ');
|
||||
if (wizards.length === 1) {
|
||||
wizards[0][property] = newValue;
|
||||
} else {
|
||||
throw 'non unique wizard, unable to set ' + property;
|
||||
}
|
||||
}
|
||||
},
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user