public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH docs/manager/qemu-server 0/6] fix #7213: add `powercycle` reboot behavior
@ 2026-08-13 11:27 Jakob Klocker
  2026-08-13 11:27 ` [PATCH qemu-server 1/6] qm: do not restart VM when 'reboot' is disabled Jakob Klocker
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Jakob Klocker @ 2026-08-13 11:27 UTC (permalink / raw)
  To: pve-devel; +Cc: Jakob Klocker

A reboot initiated inside the guest is handled by QEMU itself: the 
process keeps running and only the guest OS boots again. That is fast,
but the VM stays on the configuration and QEMU binary it was started
with, so pending changes remain pending.

This series adds a `powercycle` sub-property to the existing `reboot`
option. With it set, the VM is started with `-no-reboot`, so QEMU exits
on a reset; qmeventd reports the reason, and `qm cleanup` starts
the VM again. The result is a full start, so pending changes take
effect.

To carry the new sub-property, `reboot` is converted to a property
string. Existing configurations stay valid, as the previous boolean
value is the default key.

Note two behavior changes beyond the new sub-property: `reboot: 0` now
also prevents a restart for reboots triggered through the web interface,
`qm` or the API - previously only guest-initiated reboots were affected.
The documented meaning of the option is that the VM does not reboot, so
this seemed like the more consistent reading. And with `powercycle` set,
a reset requested through the web interface, `qm` or the API also becomes
a full stop and start, since `-no-reboot` makes QEMU exit on any reset.

Link: https://bugzilla.proxmox.com/show_bug.cgi?id=7213


qemu-server:

Jakob Klocker (4):
  qm: do not restart VM when 'reboot' is disabled
  fix #7213: config: add `powercycle` sub-property to `reboot`
  fix #7213: qm: cleanup: add `reset` parameter to honor `powercycle`
  fix #7213: qmeventd: pass `reset` to `qm cleanup`

 src/PVE/CLI/qm.pm       | 22 +++++++++++++++++++---
 src/PVE/QemuServer.pm   | 41 ++++++++++++++++++++++++++++++++++++-----
 src/qmeventd/qmeventd.c | 36 ++++++++++++++++++++++++++++--------
 src/qmeventd/qmeventd.h |  1 +
 4 files changed, 84 insertions(+), 16 deletions(-)


pve-manager:

Jakob Klocker (1):
  ui: qemu: options: add editor for reboot behavior

 www/manager6/Makefile                      |  1 +
 www/manager6/Utils.js                      | 17 +++++++
 www/manager6/form/RebootFeatureSelector.js | 59 ++++++++++++++++++++++
 www/manager6/qemu/Options.js               | 14 +++++
 4 files changed, 91 insertions(+)
 create mode 100644 www/manager6/form/RebootFeatureSelector.js


pve-docs:

Jakob Klocker (1):
  qm: add reboot behavior information

 qm.adoc | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)


Summary over all repositories:
  9 files changed, 212 insertions(+), 16 deletions(-)

-- 
Generated by murpp 0.12.0



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2026-08-13 11:28 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-13 11:27 [PATCH docs/manager/qemu-server 0/6] fix #7213: add `powercycle` reboot behavior Jakob Klocker
2026-08-13 11:27 ` [PATCH qemu-server 1/6] qm: do not restart VM when 'reboot' is disabled Jakob Klocker
2026-08-13 11:27 ` [PATCH qemu-server 2/6] fix #7213: config: add `powercycle` sub-property to `reboot` Jakob Klocker
2026-08-13 11:27 ` [PATCH qemu-server 3/6] fix #7213: qm: cleanup: add `reset` parameter to honor `powercycle` Jakob Klocker
2026-08-13 11:27 ` [PATCH qemu-server 4/6] fix #7213: qmeventd: pass `reset` to `qm cleanup` Jakob Klocker
2026-08-13 11:27 ` [PATCH pve-manager 5/6] ui: qemu: options: add editor for reboot behavior Jakob Klocker
2026-08-13 11:27 ` [PATCH pve-docs 6/6] qm: add reboot behavior information Jakob Klocker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal