all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Hannes Duerr <h.duerr@proxmox.com>
To: Proxmox Datacenter Manager development discussion
	<pdm-devel@lists.proxmox.com>,
	Stefan Hanreich <s.hanreich@proxmox.com>
Subject: Re: [pdm-devel] [PATCH proxmox{, -datacenter-manager} 0/6] Add support for network resource type.
Date: Wed, 12 Nov 2025 17:15:45 +0100	[thread overview]
Message-ID: <d7da920a-2b08-403d-8df5-00700d4146c7@proxmox.com> (raw)
In-Reply-To: <20251107144018.700695-1-s.hanreich@proxmox.com>

Thank you for looking into this.
I set up two clusters and tested the fabric integration with OSPF and 
openfabric respectively.
I also created a Simple, VLAN Zone, and VXLAN Zone, looked at the UI 
changes in each case, and the bridge output looks good.

I did not come across any issues.

Please consider:

Tested-by: Hannes Dürr <h.duerr@proxmox.com>

On 11/7/25 3:40 PM, Stefan Hanreich wrote:
> The patch series [1] introduced a new resource type for network entities. This
> patch series implements support for handling this new resource type in the
> backend, as well as the frontend. The backend can now handle both formats and
> maps them to the same Resource internally.
>
> Without this patch series applied, PDM cannot handle any Proxmox VE remotes,
> that have the patch series in [1] applied.
>
> A new match category is also introduced, in order to be able to filter for
> specific network types in the UI / API.
>
> For more information about the new resource type see the respective patch
> series [1].
>
> pre-build packages available on sani: `network-resources-pdm`
>
> [1] https://lore.proxmox.com/pve-devel/20251107143201.689035-1-s.hanreich@proxmox.com/T/
>
> proxmox:
>
> Stefan Hanreich (3):
>    pve-api-types: update /cluster/resources endpoint
>    pve-api-types: regenerate
>    pve-api-types: extend ClusterResourceNetworkType
>
>   pve-api-types/pve-api.json           | 27 +++++++++++++++-
>   pve-api-types/src/generated/types.rs | 48 ++++++++++++++++++++++++++++
>   pve-api-types/src/sdn.rs             | 11 ++++++-
>   3 files changed, 84 insertions(+), 2 deletions(-)
>
>
> proxmox-datacenter-manager:
>
> Stefan Hanreich (3):
>    resources: support new pve network resource type
>    resources: api: add network type match category
>    ui: use new network resource type
>
>   cli/client/src/resources.rs                  |   9 +-
>   lib/pdm-api-types/src/resource.rs            | 115 +++++++++++-----
>   lib/pdm-client/src/lib.rs                    |   2 +
>   server/src/api/resources.rs                  |  98 ++++++++++---
>   server/src/metric_collection/top_entities.rs |   2 +-
>   ui/src/dashboard/sdn_zone_panel.rs           |   7 +-
>   ui/src/lib.rs                                |   6 +-
>   ui/src/pve/remote.rs                         |   2 +-
>   ui/src/pve/tree.rs                           |   2 +-
>   ui/src/pve/utils.rs                          |  14 +-
>   ui/src/renderer.rs                           |   7 +-
>   ui/src/sdn/zone_tree.rs                      | 136 +++++++++++--------
>   12 files changed, 274 insertions(+), 126 deletions(-)
>
>
> Summary over all repositories:
>    15 files changed, 358 insertions(+), 128 deletions(-)
>


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

  parent reply	other threads:[~2025-11-12 16:15 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-07 14:40 Stefan Hanreich
2025-11-07 14:40 ` [pdm-devel] [PATCH proxmox 1/3] pve-api-types: update /cluster/resources endpoint Stefan Hanreich
2025-11-12 21:00   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-07 14:40 ` [pdm-devel] [PATCH proxmox 2/3] pve-api-types: regenerate Stefan Hanreich
2025-11-12 21:00   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-07 14:40 ` [pdm-devel] [PATCH proxmox 3/3] pve-api-types: extend ClusterResourceNetworkType Stefan Hanreich
2025-11-12 21:00   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-07 14:40 ` [pdm-devel] [PATCH proxmox-datacenter-manager 1/3] resources: support new pve network resource type Stefan Hanreich
2025-11-07 14:40 ` [pdm-devel] [PATCH proxmox-datacenter-manager 2/3] resources: api: add network type match category Stefan Hanreich
2025-11-07 14:40 ` [pdm-devel] [PATCH proxmox-datacenter-manager 3/3] ui: use new network resource type Stefan Hanreich
2025-11-12 16:15 ` Hannes Duerr [this message]
2025-11-13 16:24 ` [pdm-devel] superseded: [PATCH proxmox{, -datacenter-manager} 0/6] Add support for " 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=d7da920a-2b08-403d-8df5-00700d4146c7@proxmox.com \
    --to=h.duerr@proxmox.com \
    --cc=pdm-devel@lists.proxmox.com \
    --cc=s.hanreich@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.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal