[pve-devel] [PATCH manager] parser: qemu cpu: also allow kebab-cased properties while parsing

[pve-devel] [PATCH manager] parser: qemu cpu: also allow kebab-cased properties while parsing

[Daniel Kral]

There are quite a few kebab-cased properties in the QEMU CPU config,
such as phys-bits and guest-phys-bits. These are currently not exposed
through the web interface, but only the command line.

If the QEMU CPU config is parsed, it will return undefined with an
error and will break the ProcessorEdit component so that changes cannot
be submitted anymore.

Fix that by allowing kebab-cased properties as well.

Signed-off-by: Daniel Kral <d.kral@proxmox.com>
---
 www/manager6/Parser.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/manager6/Parser.js b/www/manager6/Parser.js
index 4e0f530b..9b68434d 100644
--- a/www/manager6/Parser.js
+++ b/www/manager6/Parser.js
@@ -533,7 +533,7 @@ Ext.define('PVE.Parser', {
                     return undefined; // continue
                 }
 
-                let match = p.match(/^([a-z_]+)=(\S+)$/);
+                let match = p.match(/^([a-z_\-]+)=(\S+)$/);
                 if (!match || Ext.isDefined(res[match[1]])) {
                     errors = true;
                     return false; // break
-- 
2.47.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH manager] parser: qemu cpu: also allow kebab-cased properties while parsing

[Thomas Lamprecht]

On Fri, 05 Sep 2025 16:22:21 +0200, Daniel Kral wrote:
> There are quite a few kebab-cased properties in the QEMU CPU config,
> such as phys-bits and guest-phys-bits. These are currently not exposed
> through the web interface, but only the command line.
> 
> If the QEMU CPU config is parsed, it will return undefined with an
> error and will break the ProcessorEdit component so that changes cannot
> be submitted anymore.
> 
> [...]

Applied with squashing in a fixup to make the biome linter happy again, as
escaping the - as written was not required, while this is just a warning when
installing directly, we turn warnings into errors when building an actual
debian package, thanks!

[1/1] parser: qemu cpu: also allow kebab-cased properties while parsing
      commit: 2ddc4d864b59ed93b43b7bf2581b3eac32e8bfac


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel