superseded: [PATCH qemu-server v2] api: qmclone: print a message with new VMID

[Maximiliano Sandoval <m.sandoval@proxmox.com>]

Fiona Ebner <f.ebner@proxmox.com> writes:

> Am 16.03.26 um 11:13 AM schrieb Maximiliano Sandoval:
>> When doing a linked clone of a template, the task log, journal, nor the
>> UPID of the task will log the VMID of the newly created VM.
>> 
>> We add a log message at the start of the clone. This affects both full
>> and linked clones.
>
> Could you send a patch for containers too to keep it consistent?
>
>> diff --git a/src/PVE/API2/Qemu.pm b/src/PVE/API2/Qemu.pm
>> index 1e226f6f..0587ff97 100644
>> --- a/src/PVE/API2/Qemu.pm
>> +++ b/src/PVE/API2/Qemu.pm
>> @@ -4487,6 +4487,9 @@ __PACKAGE__->register_method({
>>          my $clonefn = sub {
>>              my ($conffile, $newconf, $oldconf, $vollist, $drives, $fullclone) =
>>                  $load_and_check->();
>> +
>> +            print("creating a clone of $vmid with ID $newid\n");
>
> I think we should put "of VM $vmid" rather than just the ID like we do
> in other messages.
>
>> +
>>              my $storecfg = PVE::Storage::config();
>>  
>>              # auto generate a new uuid

Superseded-by: https://lore.proxmox.com/all/20260316123820.333308-1-m.sandoval@proxmox.com/T/#u

-- 
Maximiliano