public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH manager] ui: qemu: options: use 'first disk' for legacy 'c' boot setting
@ 2022-09-28  7:54 Fiona Ebner
  2022-11-16 19:39 ` [pve-devel] applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Fiona Ebner @ 2022-09-28  7:54 UTC (permalink / raw)
  To: pve-devel

The kvm man page mentions that using 'c' will try booting from the
first hard disk, so the current '(no bootdisk)' text in the UI is not
accurate, and boot can still succeed.

Reported in the community forum:
https://forum.proxmox.com/threads/115800/

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---

An alternative would be using 'no explicit bootdisk'.

 www/manager6/qemu/Options.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/manager6/qemu/Options.js b/www/manager6/qemu/Options.js
index a1def4bb..7b112400 100644
--- a/www/manager6/qemu/Options.js
+++ b/www/manager6/qemu/Options.js
@@ -121,7 +121,7 @@ Ext.define('PVE.qemu.Options', {
 			    if (bootdisk) {
 				text += bootdisk;
 			    } else {
-				text += gettext('(no bootdisk)');
+				text += gettext('first disk');
 			    }
 			} else if (sel === 'n') {
 			    text += gettext('any net');
-- 
2.30.2





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

* [pve-devel] applied: [PATCH manager] ui: qemu: options: use 'first disk' for legacy 'c' boot setting
  2022-09-28  7:54 [pve-devel] [PATCH manager] ui: qemu: options: use 'first disk' for legacy 'c' boot setting Fiona Ebner
@ 2022-11-16 19:39 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2022-11-16 19:39 UTC (permalink / raw)
  To: Proxmox VE development discussion, Fiona Ebner

Am 28/09/2022 um 09:54 schrieb Fiona Ebner:
> The kvm man page mentions that using 'c' will try booting from the
> first hard disk, so the current '(no bootdisk)' text in the UI is not
> accurate, and boot can still succeed.
> 
> Reported in the community forum:
> https://forum.proxmox.com/threads/115800/
> 
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
> 
> An alternative would be using 'no explicit bootdisk'.
> 
>  www/manager6/qemu/Options.js | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>

applied, thanks!




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

end of thread, other threads:[~2022-11-16 19:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-28  7:54 [pve-devel] [PATCH manager] ui: qemu: options: use 'first disk' for legacy 'c' boot setting Fiona Ebner
2022-11-16 19:39 ` [pve-devel] applied: " Thomas Lamprecht

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