all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH manager] sdn: remove fabrics property correctly from EVPN controller
@ 2025-08-21  8:50 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:50 UTC (permalink / raw)
  To: pve-devel

The missing `deleteEmtpy` attribute caused the fabric property to never
be removed from the EVPN controller. 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.

Signed-off-by: Gabriel Goller <g.goller@proxmox.com>
---
 www/manager6/sdn/controllers/EvpnEdit.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/www/manager6/sdn/controllers/EvpnEdit.js b/www/manager6/sdn/controllers/EvpnEdit.js
index cf879cda23ac..67c41f476f29 100644
--- a/www/manager6/sdn/controllers/EvpnEdit.js
+++ b/www/manager6/sdn/controllers/EvpnEdit.js
@@ -32,6 +32,7 @@ Ext.define('PVE.sdn.controllers.EvpnInputPanel', {
                 displayField: 'iface',
                 fieldLabel: 'SDN Fabric',
                 allowBlank: true,
+                deleteEmpty: true,
                 skipEmptyText: true,
                 autoSelect: false,
                 emptyText: gettext('used as underlay network'),
-- 
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 EVPN controller
  2025-08-21  8:50 [pve-devel] [PATCH manager] sdn: remove fabrics property correctly from EVPN controller 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:50:06 +0200, Gabriel Goller wrote:
> The missing `deleteEmtpy` attribute caused the fabric property to never
> be removed from the EVPN controller. 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 EVPN controller
      commit: 146cb84d7ca65016d2d1a1e427e659ec769d0a95


_______________________________________________
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:07 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:50 [pve-devel] [PATCH manager] sdn: remove fabrics property correctly from EVPN controller Gabriel Goller
2025-09-05 19:07 ` [pve-devel] applied: " Thomas Lamprecht

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