[pve-devel] [PATCH manager 00/10] Display the SCSI controller type when adding a new drive

Emmanuel Kasper e.kasper at proxmox.com
Wed Sep 6 10:15:30 CEST 2017


Since we switched to the SCSI controller in the Wizard as the recommended controller
for hard disks (using VirtIO scsi in the backend) it was not clear for users
that they were using a VirtIO driver for the disk.
An example of this would be:
https://forum.proxmox.com/threads/proxmox-hypervisor-scsi-or-virtio.36506/#post-179056
This came in the trainings as well.

Example before / after:
Before: http://imgur.com/a/Ozy6v
After: http://imgur.com/a/axJBa

This unfortunately does not play well with the controller selection based on the Iothread
checkbox, as the Bus picker list would then needed to be updated based on a checkbox selection.
IMHO, it is more important to provide a good visual feedback on which elements the user is 
manipulating, so I reverted this change, which was used only in the wizard anyway.

About the patch serie:
 * 01-06 is some refactoring work to prepare for the new combo box
 * 07 use the new combobox when adding a disk
 * 08-10 use the new combobox when adding a disk in the wizard

Emmanuel Kasper (10):
  Do not change the SCSI controller type when iothread was selected
  Move Wizard static properties to class body
  Do not use 'autoselect' as a boolean when preselecting a bus
  Remove unused noScsi parameter
  Save the components we found via down() in a variable for later reuse.
  Use the noVirtiIO ControllerSelector property to build the list of
    controllers
  Enhance BusStypeSelector with the SCSI controller description
  Improve the display names of VirtIO-blk and VirtIO SCSI controllers
  Make room in QemuWizard and Edit Window so that the larger combo boxes
    properly fit.
  Propagate the SCSI hw to the Hard Disk panel when selecting an OS

 www/manager6/Utils.js                   |  4 +-
 www/manager6/form/BusTypeSelector.js    | 56 ++++++++++++++++++++++------
 www/manager6/form/ControllerSelector.js | 66 ++++++++++++++++-----------------
 www/manager6/qemu/CDEdit.js             |  3 ++
 www/manager6/qemu/CreateWizard.js       |  3 +-
 www/manager6/qemu/HDEdit.js             | 12 +++---
 www/manager6/qemu/OSTypeEdit.js         | 11 ++++--
 www/manager6/window/Edit.js             |  3 ++
 www/manager6/window/Wizard.js           | 16 ++++----
 9 files changed, 107 insertions(+), 67 deletions(-)

-- 
2.11.0





More information about the pve-devel mailing list