public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Fiona Ebner <f.ebner@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [PATCH-SERIES qemu-server 0/8] fix #7383: agent: fsfreeze: skip freeze if already frozen
Date: Tue, 24 Mar 2026 14:50:40 +0100	[thread overview]
Message-ID: <20260324135325.120749-1-f.ebner@proxmox.com> (raw)

The series is structured as follows:

The first patch is an independent fix for a long-standing issue that
the result of the guest-fsfreeze-thaw command was not checked.

The next two patches fix checking whether the agent is enabled for
backup and clone/import, which in practice avoids a warning about the
agent not running if explicitly disabled.

The next four patches lead up to harmonizing the checks before freeze
which previously were duplicated to three different places and improve
encapsulation of the agent module.

The last patch closes bug #7383 and checks if the filesystem is
already frozen before attempting freeze to avoid a confusing error
message.

qemu-server:

Fiona Ebner (8):
  agent: fs thaw: check command result
  api: clone/import: fix check if agent is enabled
  backup: freeze: fix check if agent is enabled
  agent: parse: change signature to take property string rather than
    full VM config
  agent: get qga key: change signature to take property string rather
    than full VM config
  clone disk/block jobs: change signatures to take guest agent property
    string
  agent: fsfreeze: harmonize checks for guest fs freeze
  fix #7383: agent: fsfreeze: skip freeze if already frozen

 src/PVE/API2/Qemu.pm           | 15 ++----
 src/PVE/QemuConfig.pm          | 14 ++----
 src/PVE/QemuMigrate.pm         |  2 +-
 src/PVE/QemuServer.pm          | 14 ++++--
 src/PVE/QemuServer/Agent.pm    | 89 +++++++++++++++++++++++++++++++---
 src/PVE/QemuServer/BlockJob.pm | 15 ++----
 src/PVE/VZDump/QemuServer.pm   | 20 +++-----
 7 files changed, 114 insertions(+), 55 deletions(-)


Summary over all repositories:
  7 files changed, 114 insertions(+), 55 deletions(-)

-- 
Generated by git-murpp 0.5.0




             reply	other threads:[~2026-03-24 13:54 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-24 13:50 Fiona Ebner [this message]
2026-03-24 13:50 ` [PATCH qemu-server 1/8] agent: fs thaw: check command result Fiona Ebner
2026-03-25 21:20   ` applied: " Thomas Lamprecht
2026-03-24 13:50 ` [PATCH qemu-server 2/8] api: clone/import: fix check if agent is enabled Fiona Ebner
2026-03-25 21:20   ` applied: " Thomas Lamprecht
2026-03-24 13:50 ` [PATCH qemu-server 3/8] backup: freeze: " Fiona Ebner
2026-03-25 21:20   ` applied: " Thomas Lamprecht
2026-03-24 13:50 ` [PATCH qemu-server 4/8] agent: parse: change signature to take property string rather than full VM config Fiona Ebner
2026-03-24 13:50 ` [PATCH qemu-server 5/8] agent: get qga key: " Fiona Ebner
2026-03-24 13:50 ` [PATCH qemu-server 6/8] clone disk/block jobs: change signatures to take guest agent property string Fiona Ebner
2026-03-24 13:50 ` [PATCH qemu-server 7/8] agent: fsfreeze: harmonize checks for guest fs freeze Fiona Ebner
2026-03-25 21:16   ` Thomas Lamprecht
2026-03-24 13:50 ` [PATCH qemu-server 8/8] fix #7383: agent: fsfreeze: skip freeze if already frozen Fiona Ebner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260324135325.120749-1-f.ebner@proxmox.com \
    --to=f.ebner@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal