Re: [PATCH docs/manager/qemu-server v4 00/17] Add API and UI for custom CPU models

Arthur Bied-Charreton
2026-05-07T07:15:30Z

Hey David, thanks a lot for testing!

On Wed, May 06, 2026 at 04:31:05PM +0200, David Riley wrote:
> Thanks for the patch. This seems to be a nice addition.
>
> I’ve tested the series and found a regression in the VM creation
> wizard. When creating a Windows guest with the 'Add additional drive
> for VirtIO drivers' checkbox enabled and an ISO Image is added,
> the creation fails with:
>
> 'Parameter verification failed. (400) ide0:
> explicit media=cdrom is required for iso images'
>
> Looking at the request payload, the 'media=cdrom' property is missing
> for the VirtIO ISO:
>
>   Before: ide0: "sani:iso/virtio-win-0.1.271.iso,media=cdrom"
>   Now: ide0: "sani:iso/virtio-win-0.1.271.iso"
>
I was able to reproduce this on a fresh PVE VM with pve-manager 9.1.10.
The culprit is commit f196d28f on pve-manager, which moved the ISO
selector to a new panel, but omitted the onGetValues hook that was
adding media=cdrom. I confirmed the issue with Dominik and sent a patch
fixing it [0].

> I also noticed that the wizard now defaults to the IDE bus for disks
> instead of SCSI, when the Guest OS Type Microsoft Windows is selected.
>
This came with commit 6be73bfe on pve-manager, which made some changes
to the sorting priority of that field. I think this is intentional?
Maybe @Dominik can confirm.
>
> On 4/30/26 5:59 PM, Arthur Bied-Charreton wrote:
[...]

[0] https://lore.proxmox.com/pve-devel/20260507065517.237996-1-a.bied-charreton@proxmox.com/T/#u