[PATCH manager] ui: qemu: boot order: warn of missing pxe support for ovmf

[PATCH manager] ui: qemu: boot order: warn of missing pxe support for ovmf

[Dominik Csapak]

OVMF can only use PXE/network boot when a virtual RNG device is added
for VMs, but this is not obvious from the UI or the error behavior.

Add a small warning to the boot order edit that says what is necessary.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
I guess we don't want to add warnings for every little thing, but this
is very non-obvious when one needs its, since ovmf simply does not show
the network device, and adding an RNG device would not really come to
mind...

To be fair, it is written in the docs, but I think a short warning here
wouldn't hurt...

 www/manager6/qemu/BootOrderEdit.js | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/www/manager6/qemu/BootOrderEdit.js b/www/manager6/qemu/BootOrderEdit.js
index 521a3d6e..c577c2c2 100644
--- a/www/manager6/qemu/BootOrderEdit.js
+++ b/www/manager6/qemu/BootOrderEdit.js
@@ -26,6 +26,7 @@ Ext.define('PVE.qemu.BootOrderPanel', {
             let grid = me.lookup('grid');
             let marker = me.lookup('marker');
             let emptyWarning = me.lookup('emptyWarning');
+            let rngPxeWarning = me.lookup('rngPxeWarning');
 
             marker.originalValue = undefined;
 
@@ -43,6 +44,19 @@ Ext.define('PVE.qemu.BootOrderPanel', {
                         view.inUpdate = false;
                         marker.checkDirty();
                         emptyWarning.setHidden(val !== '');
+
+                        let showRngPxeWarning = false;
+                        if (
+                            view.vmconfig?.bios?.toLowerCase().includes('ovmf') &&
+                            !view.vmconfig?.rng0 &&
+                            val?.includes('net')
+                        ) {
+                            showRngPxeWarning = true;
+                        }
+
+                        console.log('pxe?', showRngPxeWarning);
+                        rngPxeWarning.setHidden(!showRngPxeWarning);
+
                         grid.getView().refresh();
                     },
                 },
@@ -238,6 +252,15 @@ Ext.define('PVE.qemu.BootOrderPanel', {
             userCls: 'pmx-hint',
             value: gettext('Warning: No devices selected, the VM will probably not boot!'),
         },
+        {
+            xtype: 'displayfield',
+            reference: 'rngPxeWarning',
+            userCls: 'pmx-hint',
+            hidden: true,
+            value: gettext(
+                'Warning: PXE boot with OVMF only works when a Virtio RNG device is added!',
+            ),
+        },
         {
             // for dirty marking and 'reset' function
             xtype: 'field',
-- 
2.47.3

superseded: [PATCH manager] ui: qemu: boot order: warn of missing pxe support for ovmf

[Dominik Csapak]

superseded by v2:
https://lore.proxmox.com/pve-devel/20260609121144.2627513-1-d.csapak@proxmox.com/T/#u