public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH manager] sdn: remove fabrics property correctly from the Vxlan Zone
@ 2025-08-21  8:34 Gabriel Goller
  2025-09-05 19:07 ` [pve-devel] applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Gabriel Goller @ 2025-08-21  8:34 UTC (permalink / raw)
  To: pve-devel

The missing `deleteEmtpy` attribute caused the fabric property to never
be removed. So when e.g. removing the fabric to add the peers manually,
there would always be an error because you can't have peers *and*
fabrics configured.

Fixes: https://bugzilla.proxmox.com/show_bug.cgi?id=6710
Signed-off-by: Gabriel Goller <g.goller@proxmox.com>
---
 www/manager6/sdn/zones/VxlanEdit.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/www/manager6/sdn/zones/VxlanEdit.js b/www/manager6/sdn/zones/VxlanEdit.js
index 20e1c5bb7b00..43494deca233 100644
--- a/www/manager6/sdn/zones/VxlanEdit.js
+++ b/www/manager6/sdn/zones/VxlanEdit.js
@@ -54,6 +54,7 @@ Ext.define('PVE.sdn.zones.VxlanInputPanel', {
                 fieldLabel: 'SDN Fabric',
                 skipEmptyText: true,
                 allowBlank: true,
+                deleteEmpty: true,
                 autoSelect: false,
                 emptyText: gettext('used as underlay network'),
                 nodename: 'localhost',
-- 
2.47.2



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


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

* [pve-devel] applied: [PATCH manager] sdn: remove fabrics property correctly from the Vxlan Zone
  2025-08-21  8:34 [pve-devel] [PATCH manager] sdn: remove fabrics property correctly from the Vxlan Zone Gabriel Goller
@ 2025-09-05 19:07 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2025-09-05 19:07 UTC (permalink / raw)
  To: pve-devel, Gabriel Goller

On Thu, 21 Aug 2025 10:34:53 +0200, Gabriel Goller wrote:
> The missing `deleteEmtpy` attribute caused the fabric property to never
> be removed. So when e.g. removing the fabric to add the peers manually,
> there would always be an error because you can't have peers *and*
> fabrics configured.
> 
> 

Applied, thanks!

[1/1] sdn: remove fabrics property correctly from the Vxlan Zone
      commit: 1b37bf06a0e83a6ace1ad250ef795048d65475e8


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


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

end of thread, other threads:[~2025-09-05 19:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-21  8:34 [pve-devel] [PATCH manager] sdn: remove fabrics property correctly from the Vxlan Zone Gabriel Goller
2025-09-05 19:07 ` [pve-devel] applied: " Thomas Lamprecht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal