all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Stefan Reiter <s.reiter@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH v2 0/7] Improve boot device/order configuration
Date: Tue,  6 Oct 2020 15:32:11 +0200	[thread overview]
Message-ID: <20201006133218.25403-1-s.reiter@proxmox.com> (raw)

The goal of the series is to modernize boot order selection by allowing the user
to specify arbitrary VM disks and devices in any order. Fixes #3010 and #3011.

First patch is a standalone cleanup. Tests are seperated into extra patches.

v2:
* split qemu-server changes into seperate patches (hence changelog only here)
* use order= property in boot: instead of standalone bootorder:
** update API and GUI to match
* fix get_default_bootdevices and bootorder_from_legacy to *actually* match the
old behaviour, as is now verified with the test
=> see GUI patch for changes there


qemu-server: Stefan Reiter (6):
  fix indentation
  cfg2cmd: add test for legacy-style bootorder
  add new 'boot' property format and introduce legacy conversion helpers
  fix #3010: add 'bootorder' parameter for better control of boot
    devices
  api: add handling for new boot order format
  cfg2cmd: add tests for new boot order property

 PVE/API2/Qemu.pm                       |  37 ++++-
 PVE/CLI/qm.pm                          |   4 +-
 PVE/QemuServer.pm                      | 207 ++++++++++++++++++++-----
 PVE/QemuServer/Drive.pm                |  32 +++-
 PVE/QemuServer/PCI.pm                  |   3 +-
 PVE/QemuServer/USB.pm                  |  14 +-
 test/cfg2cmd/bootorder-empty.conf      |  16 ++
 test/cfg2cmd/bootorder-empty.conf.cmd  |  38 +++++
 test/cfg2cmd/bootorder-legacy.conf     |  17 ++
 test/cfg2cmd/bootorder-legacy.conf.cmd |  38 +++++
 test/cfg2cmd/bootorder.conf            |  16 ++
 test/cfg2cmd/bootorder.conf.cmd        |  38 +++++
 12 files changed, 398 insertions(+), 62 deletions(-)
 create mode 100644 test/cfg2cmd/bootorder-empty.conf
 create mode 100644 test/cfg2cmd/bootorder-empty.conf.cmd
 create mode 100644 test/cfg2cmd/bootorder-legacy.conf
 create mode 100644 test/cfg2cmd/bootorder-legacy.conf.cmd
 create mode 100644 test/cfg2cmd/bootorder.conf
 create mode 100644 test/cfg2cmd/bootorder.conf.cmd

manager: Stefan Reiter (1):
  ui: improve boot order editor with 'bootorder' support

 www/css/ext6-pve.css               |   4 +
 www/manager6/qemu/BootOrderEdit.js | 355 ++++++++++++++++++-----------
 www/manager6/qemu/Options.js       |  32 ++-
 3 files changed, 257 insertions(+), 134 deletions(-)

-- 
2.20.1




             reply	other threads:[~2020-10-06 13:33 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-06 13:32 Stefan Reiter [this message]
2020-10-06 13:32 ` [pve-devel] [PATCH v2 qemu-server 1/7] fix indentation Stefan Reiter
2020-10-06 13:32 ` [pve-devel] [PATCH v2 qemu-server 2/7] cfg2cmd: add test for legacy-style bootorder Stefan Reiter
2020-10-06 13:32 ` [pve-devel] [PATCH v2 qemu-server 3/7] add new 'boot' property format and introduce legacy conversion helpers Stefan Reiter
2020-10-06 13:32 ` [pve-devel] [PATCH v2 qemu-server 4/7] fix #3010: add 'bootorder' parameter for better control of boot devices Stefan Reiter
2020-10-16 14:53   ` Thomas Lamprecht
2020-10-06 13:32 ` [pve-devel] [PATCH v2 qemu-server 5/7] api: add handling for new boot order format Stefan Reiter
2020-10-06 13:32 ` [pve-devel] [PATCH v2 qemu-server 6/7] cfg2cmd: add tests for new boot order property Stefan Reiter
2020-10-06 13:32 ` [pve-devel] [PATCH v2 manager 7/7] ui: improve boot order editor Stefan Reiter
2020-10-16 12:50 ` [pve-devel] applied-series: [PATCH v2 0/7] Improve boot device/order configuration Thomas Lamprecht

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201006133218.25403-1-s.reiter@proxmox.com \
    --to=s.reiter@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal