From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id 332511FF13F for ; Thu, 23 Apr 2026 14:40:39 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 7CCF316929; Thu, 23 Apr 2026 14:40:33 +0200 (CEST) From: Fiona Ebner To: pve-devel@lists.proxmox.com Subject: [PATCH qemu-server v2 7/7] fix #7383: agent: fsfreeze: skip freeze if already frozen Date: Thu, 23 Apr 2026 14:35:18 +0200 Message-ID: <20260423124004.115303-8-f.ebner@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260423124004.115303-1-f.ebner@proxmox.com> References: <20260423124004.115303-1-f.ebner@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1776947919298 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.008 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: HD3ALNNJO42D537DZQ5X3C2D2DS23BDA X-Message-ID-Hash: HD3ALNNJO42D537DZQ5X3C2D2DS23BDA X-MailFrom: f.ebner@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: This avoids the following error message about the command being disallowed which can be hard to interpret for users: > Command guest-fsfreeze-freeze has been disabled: the command is not allowed It's disallowed, because the fs is already frozen, but the error message does not indicate this. Instead avoid the error by skipping if already frozen and log it if that is the case. Signed-off-by: Fiona Ebner --- Changes in v2: * Use subroutine signature. * Adapt to log function change. src/PVE/QemuServer/Agent.pm | 31 ++++++++++++++++++++++++++++--- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/src/PVE/QemuServer/Agent.pm b/src/PVE/QemuServer/Agent.pm index c8cd334f..be6df443 100644 --- a/src/PVE/QemuServer/Agent.pm +++ b/src/PVE/QemuServer/Agent.pm @@ -276,6 +276,23 @@ sub guest_fs_thaw($vmid) { return; } +=head3 guest_fs_is_frozen + + if (guest_fs_is_frozen($vmid)) { + print "skipping guest fs freeze - already frozen\n"; + } + +Check if the file systems of the guest C<$vmid> is currently frozen. + +=cut + +sub guest_fs_is_frozen($vmid) { + my $status = PVE::QemuServer::Monitor::mon_cmd($vmid, "guest-fsfreeze-status"); + check_agent_error($status, "unable to check guest filesystem freeze status"); + + return $status eq 'frozen'; +} + =head3 guest_fs_freeze_applicable if (guest_fs_freeze_applicable($agent_str, $vmid, $logfunc)) { @@ -283,9 +300,9 @@ sub guest_fs_thaw($vmid) { } Check if the file systems of the guest C<$vmid> should be frozen according to the guest agent -property string C<$agent_str> and if freezing is actionable in practice, i.e. guest agent running. -Logs a message if skipped. Using a custom log function via C<$logfunc> is supported. Otherwise, -C and C will be used. +property string C<$agent_str> and if freezing is actionable in practice, i.e. guest agent running +and file system not already frozen. Logs a message if skipped. Using a custom log function via +C<$logfunc> is supported. Otherwise, C and C will be used. =cut @@ -304,6 +321,14 @@ sub guest_fs_freeze_applicable($agent_str, $vmid, $logfunc = undef) { return; } + my $frozen = eval { PVE::QemuServer::Agent::guest_fs_is_frozen($vmid) }; + $logfunc->("unable to check guest fs freeze status - $@", 'warn') if $@; + + if ($frozen) { + $logfunc->("skipping guest filesystem freeze - already frozen"); + return; + } + return 1; } -- 2.47.3