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

[Dominik Csapak <d.csapak@proxmox.com>]

this is actually v3, sorry^^

On 6/10/26 3:41 PM, Dominik Csapak wrote:
> 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>
> Reviewed-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
> changes from v1:
> * incorporated fionas feedback (thanks!)
> * include r-b trailer from fiona
> 
>   www/manager6/qemu/BootOrderEdit.js | 16 ++++++++++++++++
>   1 file changed, 16 insertions(+)
> 
> diff --git a/www/manager6/qemu/BootOrderEdit.js b/www/manager6/qemu/BootOrderEdit.js
> index 521a3d6e..08e71913 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,14 @@ Ext.define('PVE.qemu.BootOrderPanel', {
>                           view.inUpdate = false;
>                           marker.checkDirty();
>                           emptyWarning.setHidden(val !== '');
> +
> +                        let showRngPxeWarning =
> +                            view.vmconfig?.bios === 'ovmf' &&
> +                            !view.vmconfig?.rng0 &&
> +                            val?.includes('net');
> +
> +                        rngPxeWarning.setHidden(!showRngPxeWarning);
> +
>                           grid.getView().refresh();
>                       },
>                   },
> @@ -238,6 +247,13 @@ 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: For PXE boot with OVMF, you must add a VirtIO RNG device!'),
> +        },
>           {
>               // for dirty marking and 'reset' function
>               xtype: 'field',