[pve-devel] [PATCH v2 manager] ui: vm network: allow to override MTU for virtio devices

Aaron Lauterer a.lauterer at proxmox.com
Thu Feb 10 15:14:43 CET 2022



On 2/10/22 15:08, Oguz Bektas wrote:

> thanks for the test and review!
> 
> instead of a validator function i guess we could just set minValue and
> maxValue as well? (since allowBlank is set to true)
> 
> seemed to work fine in my short testing just now :)
> 
> diff --git a/www/manager6/qemu/NetworkEdit.js b/www/manager6/qemu/NetworkEdit.js
> index 1e34ad1c..a3fa5724 100644
> --- a/www/manager6/qemu/NetworkEdit.js
> +++ b/www/manager6/qemu/NetworkEdit.js
> @@ -184,6 +184,8 @@ Ext.define('PVE.qemu.NetworkInputPanel', {
>   		bind: {
>   		    disabled: '{!isVirtio}',
>   		},
> +		minValue: 1,
> +		maxValue: 65520,
>   		allowBlank: true,
>   	    },
>   	];

Duh of course...
That does what we want and is much nicer :)





More information about the pve-devel mailing list