Re: [PATCH qemu-server v2 2/2] vm start: log qemu version when starting a VM

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

Am 30.06.26 um 9:04 AM schrieb Maximiliano Sandoval:
> Previously, having the system journal and task logs from a boot was not
> enough to determine which qemu version is installed nor the running QEMU
> version of a VM. This is addressed by logging pve-qemu-kvm's package
> version when starting the VM.
> 
> The package version is obtained with the 'query-version' qmp command.
> 
> Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
> ---
> 
> Notes:
>     sample output in the journal:
>     
>     Jun 24 10:39:14 pve-148 qm[26137]: VM 100 started with PID 26166 using pve-qemu-kvm_11.0.0-4.
> 
>  src/PVE/QemuServer.pm | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/src/PVE/QemuServer.pm b/src/PVE/QemuServer.pm
> index cdf66e89..3ecd16a7 100644
> --- a/src/PVE/QemuServer.pm
> +++ b/src/PVE/QemuServer.pm
> @@ -5853,7 +5853,8 @@ sub vm_start_nolock {
>      eval { PVE::QemuServer::PCI::reserve_pci_usage($pci_reserve_list, $vmid, undef, $pid) };
>      warn $@ if $@;
>  
> -    syslog("info", "VM $vmid started with PID $pid.");
> +    my $pkg_version = get_qemu_package_version($vmid);
> +    syslog("info", "VM $vmid started with PID $pid using $pkg_version.");
>  
>      PVE::QemuServer::RunState::create_cleanup_flag($vmid);
>  
> @@ -8068,6 +8069,12 @@ sub get_running_qemu_version {
>      return "$res->{qemu}->{major}.$res->{qemu}->{minor}";
>  }
>  
> +sub get_qemu_package_version {

If you want to keep it beside get_running_qemu_version(), then that one
should be moved to QMPHelpers as well. The main QemuServer.pm module is
already bloated enough. We should take the opportunities to reduce it.

> +    my ($vmid) = @_;
> +    my $res = mon_cmd($vmid, "query-version");
> +    return $res->{package};
> +}> +
>  sub qemu_use_old_bios_files {
>      my ($machine_type) = @_;
>