From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id AA12C1FF141 for ; Fri, 13 Mar 2026 10:55:45 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id CA6BED0EE; Fri, 13 Mar 2026 10:55:39 +0100 (CET) From: Maximiliano Sandoval To: pve-devel@lists.proxmox.com Subject: [PATCH manager v2 0/7] ui: qga: add checkbox for guest-fsfreeze Date: Fri, 13 Mar 2026 10:55:26 +0100 Message-ID: <20260313095534.79526-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: 1773395697178 X-SPAM-LEVEL: Spam detection results: 0 AWL -0.956 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 RCVD_IN_MSPIKE_H2 0.001 Average reputation (+2) RCVD_IN_VALIDITY_CERTIFIED_BLOCKED 0.408 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_RPBL_BLOCKED 0.819 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_SAFE_BLOCKED 0.903 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. 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: DIADOYPY6C4GNTZ72TFOWIZWCKGSPZT5 X-Message-ID-Hash: DIADOYPY6C4GNTZ72TFOWIZWCKGSPZT5 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: 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 and explicitly disabled then the old checkbox would be displayed - But we delete the deprecated key from the config if it was explicitly enabled (it is the default) after performing any change 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. Differences from v1: - Rephrased the user-facing strings - Fix a typo in a commit message - Expand some commit messages - Hide the checkbox not only if it was undefined, but also if it was explicitly set to the default value 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 conditionally www/manager6/Utils.js | 2 + www/manager6/form/AgentFeatureSelector.js | 72 ++++++++++++++++++++--- 2 files changed, 66 insertions(+), 8 deletions(-) -- 2.47.3