From: Stefan Hanreich <s.hanreich@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH pve-manager 5/8] ui: resource tree: add network resource
Date: Thu, 30 Oct 2025 16:48:39 +0100	[thread overview]
Message-ID: <20251030154851.540408-33-s.hanreich@proxmox.com> (raw)
In-Reply-To: <20251030154851.540408-1-s.hanreich@proxmox.com>
From: Gabriel Goller <g.goller@proxmox.com>
Add the newly introduced network resource to the resource tree, so the
the status of fabrics can be displayed in the UI. For this matter, a
new NetworkBrowser widget is added, which is responsible for showing
the contents of a network resource.
The NetworkBrowser widget also contains code for handling the zone
type, which is currently still contained in the sdn resource type.
This ensures a smooth transition when moving the zones into the new
network resource type.
Co-authored-by: Stefan Hanreich <s.hanreich@proxmox.com>
Signed-off-by: Gabriel Goller <g.goller@proxmox.com>
Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
---
 www/manager6/Makefile                  |   2 +
 www/manager6/Utils.js                  |  11 ++
 www/manager6/Workspace.js              |   1 +
 www/manager6/sdn/FabricsContentView.js |  77 ++++++++++++
 www/manager6/sdn/NetworkBrowser.js     | 167 +++++++++++++++++++++++++
 www/manager6/tree/ResourceTree.js      |   6 +
 6 files changed, 264 insertions(+)
 create mode 100644 www/manager6/sdn/FabricsContentView.js
 create mode 100644 www/manager6/sdn/NetworkBrowser.js
diff --git a/www/manager6/Makefile b/www/manager6/Makefile
index 85f9268d1..ba762578e 100644
--- a/www/manager6/Makefile
+++ b/www/manager6/Makefile
@@ -278,6 +278,7 @@ JSSRC= 							\
 	qemu/USBEdit.js					\
 	qemu/VirtiofsEdit.js				\
 	sdn/Browser.js					\
+	sdn/NetworkBrowser.js					\
 	sdn/ControllerView.js				\
 	sdn/Status.js					\
 	sdn/StatusView.js				\
@@ -313,6 +314,7 @@ JSSRC= 							\
 	sdn/zones/VlanEdit.js				\
 	sdn/zones/VxlanEdit.js				\
 	sdn/FabricsView.js				\
+	sdn/FabricsContentView.js			\
 	sdn/fabrics/Common.js				\
 	sdn/fabrics/InterfacePanel.js			\
 	sdn/fabrics/NodeEdit.js				\
diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
index c48ee0b25..2f812a442 100644
--- a/www/manager6/Utils.js
+++ b/www/manager6/Utils.js
@@ -1276,6 +1276,12 @@ Ext.define('PVE.Utils', {
                 // templates
                 objType = 'template';
                 status = type;
+            } else if (type === 'network') {
+                const network_type_mapping = {
+                    fabric: 'fa fa-road',
+                };
+
+                return network_type_mapping[record.network_type] ?? '';
             } else if (type === 'storage' && record.content === 'import') {
                 return 'fa fa-cloud-download';
             } else {
@@ -1299,6 +1305,11 @@ Ext.define('PVE.Utils', {
             var cls = PVE.Utils.get_object_icon_class(value, record.data);
 
             var fa = '<i class="fa-fw x-grid-icon-custom ' + cls + '"></i> ';
+
+            if (value === 'network') {
+                return fa + record.data.network_type;
+            }
+
             return fa + value;
         },
 
diff --git a/www/manager6/Workspace.js b/www/manager6/Workspace.js
index a25746582..9f1e807d5 100644
--- a/www/manager6/Workspace.js
+++ b/www/manager6/Workspace.js
@@ -250,6 +250,7 @@ Ext.define('PVE.StdWorkspace', {
                             lxc: 'pveLXCConfig',
                             storage: 'PVE.storage.Browser',
                             sdn: 'PVE.sdn.Browser',
+                            network: 'PVE.network.Browser',
                             pool: 'pvePoolConfig',
                             tag: 'pveTagConfig',
                         };
diff --git a/www/manager6/sdn/FabricsContentView.js b/www/manager6/sdn/FabricsContentView.js
new file mode 100644
index 000000000..47e8bce7f
--- /dev/null
+++ b/www/manager6/sdn/FabricsContentView.js
@@ -0,0 +1,77 @@
+Ext.define('PVE.sdn.FabricRoutesContentView', {
+    extend: 'Ext.grid.GridPanel',
+    alias: 'widget.pveSDNFabricRoutesContentView',
+
+    columns: [
+        {
+            header: gettext('Route'),
+            sortable: true,
+            dataIndex: 'route',
+            flex: 1,
+        },
+        {
+            header: gettext('Via'),
+            sortable: true,
+            dataIndex: 'via',
+            renderer: (value) => {
+                if (Ext.isArray(value)) {
+                    return value.join('<br>');
+                }
+                return value || '';
+            },
+            flex: 1,
+        },
+    ],
+});
+
+Ext.define('PVE.sdn.FabricNeighborsContentView', {
+    extend: 'Ext.grid.GridPanel',
+    alias: 'widget.pveSDNFabricNeighborsContentView',
+
+    columns: [
+        {
+            header: gettext('Neighbor'),
+            sortable: true,
+            dataIndex: 'neighbor',
+            flex: 1,
+        },
+        {
+            header: gettext('Status'),
+            sortable: true,
+            dataIndex: 'status',
+            flex: 0.5,
+        },
+        {
+            header: gettext('Uptime'),
+            sortable: true,
+            dataIndex: 'uptime',
+            flex: 0.5,
+        },
+    ],
+});
+
+Ext.define('PVE.sdn.FabricInterfacesContentView', {
+    extend: 'Ext.grid.GridPanel',
+    alias: 'widget.pveSDNFabricInterfacesContentView',
+
+    columns: [
+        {
+            header: gettext('Name'),
+            sortable: true,
+            dataIndex: 'name',
+            flex: 1,
+        },
+        {
+            header: gettext('Type'),
+            sortable: true,
+            dataIndex: 'type',
+            flex: 1,
+        },
+        {
+            header: gettext('State'),
+            sortable: true,
+            dataIndex: 'state',
+            flex: 1,
+        },
+    ],
+});
diff --git a/www/manager6/sdn/NetworkBrowser.js b/www/manager6/sdn/NetworkBrowser.js
new file mode 100644
index 000000000..ad024e6a1
--- /dev/null
+++ b/www/manager6/sdn/NetworkBrowser.js
@@ -0,0 +1,167 @@
+Ext.define('PVE.network.Browser', {
+    extend: 'PVE.panel.Config',
+    alias: 'widget.PVE.network.Browser',
+
+    initComponent: function () {
+        let me = this;
+        let data = me.pveSelNode.data;
+
+        let node = data.node;
+        if (!node) {
+            throw 'no node name specified';
+        }
+
+        let name = data.network;
+        if (!name) {
+            throw 'no name specified';
+        }
+
+        let networkType = data.network_type;
+        if (!name) {
+            throw 'no type specified';
+        }
+
+        me.items = [];
+
+        if (networkType === 'fabric') {
+            me.onlineHelp = 'pvesdn_config_fabrics';
+
+            me.items.push({
+                nodename: node,
+                fabricId: name,
+                protocol: me.pveSelNode.data.protocol,
+                xtype: 'pveSDNFabricRoutesContentView',
+                title: gettext('Routes'),
+                iconCls: 'fa fa-exchange',
+                itemId: 'routes',
+                width: '100%',
+                store: {
+                    proxy: {
+                        type: 'proxmox',
+                        url: `/api2/json/nodes/${node}/sdn/fabrics/${name}/routes`,
+                        reader: {
+                            type: 'json',
+                            rootProperty: 'data',
+                        },
+                    },
+                    autoLoad: true,
+                },
+            });
+
+            me.items.push({
+                nodename: node,
+                fabricId: name,
+                protocol: me.pveSelNode.data.protocol,
+                xtype: 'pveSDNFabricNeighborsContentView',
+                title: gettext('Neighbors'),
+                iconCls: 'fa fa-handshake-o',
+                itemId: 'neighbors',
+                width: '100%',
+                store: {
+                    proxy: {
+                        type: 'proxmox',
+                        url: `/api2/json/nodes/${node}/sdn/fabrics/${name}/neighbors`,
+                        reader: {
+                            type: 'json',
+                            rootProperty: 'data',
+                        },
+                    },
+                    autoLoad: true,
+                },
+            });
+
+            me.items.push({
+                nodename: node,
+                fabricId: name,
+                protocol: me.pveSelNode.data.protocol,
+                xtype: 'pveSDNFabricInterfacesContentView',
+                title: gettext('Interfaces'),
+                iconCls: 'fa fa-upload',
+                itemId: 'interfaces',
+                width: '100%',
+                store: {
+                    proxy: {
+                        type: 'proxmox',
+                        url: `/api2/json/nodes/${node}/sdn/fabrics/${name}/interfaces`,
+                        reader: {
+                            type: 'json',
+                            rootProperty: 'data',
+                        },
+                    },
+                    autoLoad: true,
+                },
+            });
+        } else if (networkType === 'zone') {
+            const caps = Ext.state.Manager.get('GuiCap');
+
+            me.items.push({
+                nodename: node,
+                zone: name,
+                xtype: 'pveSDNZoneContentPanel',
+                title: gettext('Content'),
+                iconCls: 'fa fa-th',
+                itemId: 'content',
+            });
+
+            if (caps.sdn['Permissions.Modify']) {
+                me.items.push({
+                    xtype: 'pveACLView',
+                    title: gettext('Permissions'),
+                    iconCls: 'fa fa-unlock',
+                    itemId: 'permissions',
+                    path: `/sdn/zones/${name}`,
+                });
+            }
+
+            me.items.push({
+                nodename: node,
+                zone: name,
+                xtype: 'pveSDNZoneBridgePanel',
+                title: gettext('Bridges'),
+                iconCls: 'fa fa-network-wired x-fa-sdn-treelist',
+                itemId: 'bridges',
+            });
+
+            if (data.zone_type && data.zone_type === 'evpn') {
+                me.items.push({
+                    nodename: node,
+                    zone: name,
+                    xtype: 'pveSDNEvpnZoneIpVrfPanel',
+                    title: gettext('IP-VRF'),
+                    iconCls: 'fa fa-th-list',
+                    itemId: 'ip-vrf',
+                });
+
+                me.items.push({
+                    nodename: node,
+                    zone: name,
+                    xtype: 'pveSDNEvpnZoneMacVrfPanel',
+                    title: gettext('MAC-VRFs'),
+                    iconCls: 'fa fa-th-list',
+                    itemId: 'mac-vrfs',
+                });
+            }
+        } else {
+            me.items.push({
+                xtype: 'container',
+                title: gettext('Content'),
+                iconCls: 'fa fa-th',
+                itemId: 'content',
+                html: `unknown network type: ${networkType}`,
+                width: '100%',
+            });
+        }
+
+        Ext.apply(me, {
+            title: Ext.String.format(
+                gettext('{0} {1} on node {2}'),
+                `${networkType}`,
+                `'${name}'`,
+                `'${node}'`,
+            ),
+            hstateid: 'networktab',
+        });
+
+        me.callParent();
+    },
+});
diff --git a/www/manager6/tree/ResourceTree.js b/www/manager6/tree/ResourceTree.js
index e83ccfc85..b6ab9a1e9 100644
--- a/www/manager6/tree/ResourceTree.js
+++ b/www/manager6/tree/ResourceTree.js
@@ -25,6 +25,10 @@ Ext.define('PVE.tree.ResourceTree', {
                 iconCls: 'fa fa-th',
                 text: gettext('SDN'),
             },
+            network: {
+                iconCls: 'fa fa-globe',
+                text: gettext('Network'),
+            },
             qemu: {
                 iconCls: 'fa fa-desktop',
                 text: gettext('Virtual Machine'),
@@ -55,6 +59,8 @@ Ext.define('PVE.tree.ResourceTree', {
                 return 2;
             case 'sdn':
                 return 3;
+            case 'network':
+                return 3.5;
             case 'storage':
                 return 4;
             default:
-- 
2.47.3
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
next prev parent reply	other threads:[~2025-10-30 15:50 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-30 15:48 [pve-devel] [PATCH common/manager/network/proxmox{-ve-rs, -perl-rs} 00/35] Improve status reporting for SDN / networking Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-common 1/2] iproute2: add helper for detecting bridge members Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-common 2/2] iproute2: add helper for querying vlan information Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-ve-rs 1/6] frr: make room for deserialization structs Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-ve-rs 2/6] frr: add deserialization types for openfabric and ospf Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-ve-rs 3/6] ve-config: add helper function to iterate over all nodes in all fabrics Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-ve-rs 4/6] ve-config: add optional tag property to vnet Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-ve-rs 5/6] frr: fix some route deserialization types Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-ve-rs 6/6] frr: add deserialization types for EVPN Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 01/10] pve-rs: firewall: cargo: fmt Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 02/10] pve-rs: firewall: add missing documentation comments Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 03/10] pve-rs: cargo: bump proxmox-apt and proxmox-ve-config versions Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 04/10] pve-rs: fabrics: update proxmox-frr import path Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 05/10] pve-rs: fabrics: fix clippy lint warnings Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 06/10] pve-rs: fabrics: add function to get status of fabric Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 07/10] pve-rs: fabrics: add function to get l2vpn and l3vpn routes for evpn Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 08/10] pve-rs: fabrics: add function to get routes learned by a fabric Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 09/10] pve-rs: fabrics: add function to get the interfaces used for " Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH proxmox-perl-rs 10/10] pve-rs: fabrics: add function to get the neighbors " Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 1/9] refactor: rework api module structure for the /nodes/{node}/sdn subdir Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 2/9] fabrics: add fabrics status to SDN::status function Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 3/9] sdn: status: add zone type to sdn resource Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 4/9] api: nodes: fabrics: add endpoint for querying route status Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 5/9] api: nodes: fabrics: add endpoint for querying neighbor information Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 6/9] api: nodes: fabrics: add endpoint for querying interface status Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 7/9] api: nodes: zones: add bridge status Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 8/9] api: nodes: zones: add ip vrf endpoint for evpn zones Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-network 9/9] api: nodes: vnets: add mac-vrf endpoint for evpn vnets Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 1/8] api: nodes: use new status module for sdn subdirectory Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 2/8] refactor: ui: sdn browser: parametrize zone content panel Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 3/8] pvestatd: add network resource to status reporting Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 4/8] pvestatd: sdn: adapt to changes in " Stefan Hanreich
2025-10-30 15:48 ` Stefan Hanreich [this message]
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 6/8] ui: sdn browser: Add ip-vrf panel for evpn zones Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 7/8] ui: sdn browser: add mac vrf panel Stefan Hanreich
2025-10-30 15:48 ` [pve-devel] [PATCH pve-manager 8/8] ui: sdn browser: add zone bridge view Stefan Hanreich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox
  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):
  git send-email \
    --in-reply-to=20251030154851.540408-33-s.hanreich@proxmox.com \
    --to=s.hanreich@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY
  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
  Be sure your reply has a Subject: header at the top and a blank line
  before the message body.
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.