[pve-devel] [PATCH v2 11/12] fix #4225: ui: qemu: cdedit: enable required checkbox for isos

[Daniel Herzig]

Daniel Kral <d.kral at proxmox.com> writes:

> On 1/13/25 09:56, Daniel Herzig wrote:
>> Enables the 'required' checkbox for the IsoSelector.
>> If the parameter is not set, the backend will use the default (set to
>> 1).
>> Behaviour:
>> * Only send parameter if not default (required=0)
>> * Checked if parameter is missing (default)
>> * Unchecked when adding a new CD-ROM
>
> IMO the part of this patch where new VMs get created with `required=0`
> CDROMs should be split into its own patch and marked as "for-9.0" or
> else add a TODO comment for the 9.0 release.
>
Good point to stay more 'conservative' for now. Thanks.