Re: [pve-devel] partially-applied: [PATCH-SERIES v12 qemu-server/manager] API for disk import and OVF

["Fabian Grünbichler" <f.gruenbichler@proxmox.com>]

On March 16, 2022 11:00 am, Fabian Ebner wrote:
> Am 14.03.22 um 16:57 schrieb Fabian Grünbichler:
>> applied qemu-server patches except 11 and 14-16, see comments on 
>> indivudal patches.
>> 
> 
> Thanks a lot for the review/feedback!
> 
>> some unrelated but possibly fix-able as followup things I noticed:
>> - cloning a running VM with an EFI disk fails, the EFI disk is not 
>>   mirrorable (so we need another check like for TPM state?)
> 
> Isn't that just when the target storage allocates a different-sized
> disk, i.e. https://bugzilla.proxmox.com/show_bug.cgi?id=3227

no, was my fault (the VM in question had an efi disk, but was not booted 
using UEFI). probably should add a check for that as well though, 
unrelated to this series (move disk is also affected, and I guess 
live-migration as well..)

>> - cancelling a running clone doesn't cleanup properly (stops with trying 
>>   to aquire lock and leaves the target VM locked & existing)
>> 
> 
> Will take a look.
> 

the exact log messages are not always the same, but the target remains 
around locked with whatever state it managed to get to (and the task 
stopped with 'unexpected status').