* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox