Re: [PATCH datacenter-manager 0/4] add resource gauge panels to dashboard/views

Lukas Wagner
2026-03-24T10:25:20Z

On Mon Mar 23, 2026 at 12:03 PM CET, Dominik Csapak wrote:
> This uses the new pie charts[0] to add gauge panels for resources to
> the dashboards/views. Either combined cpu/memory/storage or
> indidivually, for pve/pbs or combined counts.
>
> for this we have to sum the data up in the backend.
>
> I also added these to the default dashboard (since it's data from an api
> call we already query) but put that in a separate patch so we can easily
> decide to not apply that. (not sure if we want to change the default
> dashboard)
>
> Note that the pwt patches [0] have to be applied and the package
> has to be bumped first.
>
> 0: https://lore.proxmox.com/yew-devel/20260320160816.4113364-1-d.csapak@proxmox.com/
>
> Dominik Csapak (4):
> api: return global cpu/memory/storage statistics
> ui: css: use mask for svg icons
> ui: dashboard: add new gauge panels widget type
> ui: dashboard: add resource gauges to default dashboard
>
> lib/pdm-api-types/src/lib.rs | 2 +-
> lib/pdm-api-types/src/resource.rs | 27 ++++++
> lib/pdm-api-types/src/views.rs | 15 +++
> server/src/api/resources.rs | 65 ++++++++++---
> ui/css/pdm.scss | 35 +++----
> ui/src/dashboard/gauge_panel.rs | 156 ++++++++++++++++++++++++++++++
> ui/src/dashboard/mod.rs | 3 +
> ui/src/dashboard/view.rs | 19 +++-
> ui/src/dashboard/view/row_view.rs | 43 +++++++-
> 9 files changed, 331 insertions(+), 34 deletions(-)
> create mode 100644 ui/src/dashboard/gauge_panel.rs

Thanks for these patches!

Some thoughts, some of which we already discussed off-list:

- I'd suggest a wider opening angle for the gauges, so that the gauges
are a bit more compact and look less like a horse shoe
To me, 75/285 looked quite good, but this is of course highly
subjective, so no hard feelings

- For the percent-label inside the gauge, I think 0 (zero) significant
digits are okay, for such a 'global' infrastructure gauge two
significant digits are hardly useful, I think

- We probably should use full product names (Proxmox VE, Proxmox
Backup Server) in the card headers, instead of just "PVE" and "PBS".
The overall text length could become quite long though, so maybe we
need some other approach for the card title.