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 920861FF13B for ; Wed, 25 Feb 2026 16:46:01 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 8D355A4BE; Wed, 25 Feb 2026 16:46:52 +0100 (CET) From: Maximiliano Sandoval To: pve-devel@lists.proxmox.com Subject: [PATCH manager 0/7] ui: qga: add checkbox for guest-fsfreeze Date: Wed, 25 Feb 2026 16:45:39 +0100 Message-ID: <20260225154614.681858-1-m.sandoval@proxmox.com> X-Mailer: git-send-email 2.47.3 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1772034357315 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.091 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: JR2HDZER3GT3KLTMYTQQ7NGT3JH24AHF X-Message-ID-Hash: JR2HDZER3GT3KLTMYTQQ7NGT3JH24AHF X-MailFrom: m.sandoval@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 was discussed off-list with Fiona. Something that was missing from Fiona's comments was giving the "hint" css class to the "Deprecated" part of the old checkbox's label. I was not sure how to make that work in extjs. The migration would work as follows: - The new key takes preference over the old one so using the old one is OK. - If the key is in the config file then the old checkbox would dbe displayed - But delete the key from the config if the freeze is enabled, since it is the default One could additionally do some extra work where setting the new key will also set the old value, but for the sake of simplicity I think this is a good start. I added to my TODO list to add a migration check to the pve9to10 script. Maximiliano Sandoval (7): ui: utils: support guest-fsfreeze at render_qga_features ui: qga: merge translatable string ui: qga: move freeze-fs-on-backup to advanced opts ui: qga: note that freeze-fs-on-backup is deprecated ui: qga: add checkbox for guest-fsfreeze ui: qga: call parameter me instead of this ui: qga: hide freeze-fs-on-backup check if undefined www/manager6/Utils.js | 2 + www/manager6/form/AgentFeatureSelector.js | 62 ++++++++++++++++++++--- 2 files changed, 56 insertions(+), 8 deletions(-) -- 2.47.3