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

[Fiona Ebner <f.ebner@proxmox.com>]

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