[PATCH qemu-server 3/8] backup: freeze: fix check if agent is enabled

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

Just checking whether the configuration is present does not mean that
the guest agent is enabled. Explicitly check the 'enabled' property.
In case of clone/import and snapshot, this is already done. This
avoids a log message about the agent not running, since there is no
device added to the QEMU commandline for communicating with the agent
if disabled.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 src/PVE/VZDump/QemuServer.pm | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/PVE/VZDump/QemuServer.pm b/src/PVE/VZDump/QemuServer.pm
index 55fb6dc4..0681661d 100644
--- a/src/PVE/VZDump/QemuServer.pm
+++ b/src/PVE/VZDump/QemuServer.pm
@@ -1092,19 +1092,24 @@ sub _get_task_devlist {
 sub qga_fs_freeze {
     my ($self, $task, $vmid) = @_;
     return
-        if !$self->{vmlist}->{$vmid}->{agent}
-        || $task->{mode} eq 'stop'
+        if $task->{mode} eq 'stop'
         || !$self->{vm_was_running}
         || $self->{vm_was_paused};
 
+    my $conf = $self->{vmlist}->{$vmid};
+
+    if (!PVE::QemuServer::Agent::get_qga_key($conf, 'enabled')) {
+        $self->loginfo("skipping guest-agent 'fs-freeze', agent not configured in VM options");
+        return;
+    }
+
     if (!PVE::QemuServer::Agent::qga_check_running($vmid, 1)) {
         $self->loginfo("skipping guest-agent 'fs-freeze', agent configured but not running?");
         return;
     }
 
-    my $freeze = PVE::QemuServer::Agent::get_qga_key($self->{vmlist}->{$vmid}, 'guest-fsfreeze');
-    $freeze //=
-        PVE::QemuServer::Agent::get_qga_key($self->{vmlist}->{$vmid}, 'freeze-fs-on-backup');
+    my $freeze = PVE::QemuServer::Agent::get_qga_key($conf, 'guest-fsfreeze');
+    $freeze //= PVE::QemuServer::Agent::get_qga_key($conf, 'freeze-fs-on-backup');
     $freeze //= 1;
 
     if (!$freeze) {
-- 
2.47.3