all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH qemu-server] fix #7480: api: apply tag case-sensitivity setting on VM creation
@ 2026-05-22 14:42 Jakob Klocker
  2026-06-09 12:11 ` David Riley
  0 siblings, 1 reply; 2+ messages in thread
From: Jakob Klocker @ 2026-05-22 14:42 UTC (permalink / raw)
  To: pve-devel; +Cc: Jakob Klocker

The case-sensitivity setting from datacenter.cfg was not checked
when tags were set during VM creation. Tags containing capital
letters were stored verbatim, while the same tags applied later
via the update endpoint were normalized. This led to inconsistent
behavior depending on whether a tag was set at creation or
afterwards.

Link: https://bugzilla.proxmox.com/show_bug.cgi?id=7480
Signed-off-by: Jakob Klocker <j.klocker@proxmox.com>
---
 src/PVE/API2/Qemu.pm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/PVE/API2/Qemu.pm b/src/PVE/API2/Qemu.pm
index e029a204..dd813e8a 100644
--- a/src/PVE/API2/Qemu.pm
+++ b/src/PVE/API2/Qemu.pm
@@ -1500,6 +1500,10 @@ __PACKAGE__->register_method({
 
                     $conf->{lock} = 'import' if $live_import_mapping;
 
+                    if (defined $conf->{tags}) {
+                        $conf->{tags} = PVE::GuestHelpers::get_unique_tags($conf->{tags});
+                    }
+
                     PVE::QemuConfig->write_config($vmid, $conf);
                 };
                 my $err = $@;
-- 
2.47.3




^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-06-09 12:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-22 14:42 [PATCH qemu-server] fix #7480: api: apply tag case-sensitivity setting on VM creation Jakob Klocker
2026-06-09 12:11 ` David Riley

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal