Re: [pdm-devel] [PATCH proxmox{, -datacenter-manager, -yew-comp} v5 00/12] add basic integration of PVE firewall

From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Proxmox Datacenter Manager development discussion
	<pdm-devel@lists.proxmox.com>,
	Hannes Laimer <h.laimer@proxmox.com>
Subject: Re: [pdm-devel] [PATCH proxmox{, -datacenter-manager, -yew-comp} v5 00/12] add basic integration of PVE firewall
Date: Thu, 27 Nov 2025 23:01:04 +0100	[thread overview]
Message-ID: <050d8567-bc33-41b0-9d90-00c1b20c2560@proxmox.com> (raw)
In-Reply-To: <20251127092209.213535-1-h.laimer@proxmox.com>

Am 27.11.25 um 10:22 schrieb Hannes Laimer:
> Firewall options edit form and the firewall rules tables were added to
> yew-comp as they are not necesarrily PDM specific. I tried having them
> in a way so it would not be too complicated reusing them in other places
> at some point.

Nice stuff, all applied now, some things you might to look into as
potential follow-up:

Would be nice to be able to add some target web UI deep links to the remote
UI so that one gets easily there to have full edit capabilities.

Having the per-node, per-guest and datacenter levels also exposed in the
view for a specific remote would be nice, and should come almost for free.
I.e., show only one level as new tab on the right panel depending on what
is selected there in the resource tree.

Being able to add this to Views, showing only the resources available in the
view might be nice.

Being able to edit the options but not the rules is slightly odd. Maybe
disable editing the options for now too? But no hard feelings, it can have
value to be able to quickly disable or enable a FW.

A card view some basic overview/counters for the Dashboard/Views could be
nice (that one low prio though).

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