Re: [pdm-devel] [PATCH datacenter-manager v3 0/8] add task summary panels in dashboard

[Stefan Hanreich <s.hanreich@proxmox.com>]

In the review nothing major came up, and the suggestions could also be
added as follow-ups if desired.

Also gave this a quick test on my PDM instance, so consider this:

Reviewed-by: Stefan Hanreich <s.hanreich@proxmox.com>
Tested-by: Stefan Hanreich <s.hanreich@proxmox.com>

On 8/25/25 10:11 AM, Dominik Csapak wrote:
> This adds new panels to the dashboard for showing summaries of the tasks:
> * one by category
> * one by remote
> 
> similar to what we have in PBS.
> 
> I did not implement a cache for the data on the backend, but this can be
> done as a follow up and should not impact the UI code at all.
> 
> changes from v2:
> * rebase on current master (incl task caching changes from lukas)
> * new patch (8/8) that includes configuring the time frame to check the
>   tasks for
> 
> Changes from v1:
> * incorporated most suggestions from Thomas
> * split up patches a bit
> * added new server api call
> * (the UI code is very different to accomodate the necessary changes)
> 
> 
> Dominik Csapak (8):
>   server: task cache: treat a limit of 0 as unbounded
>   server: api: remote tasks: add 'remote' filter option
>   server: api: add remote-tasks statistics
>   ui: refactor remote upid formatter
>   ui: tasks: add helper to summarize task categories
>   ui: add dialog to show filtered tasks
>   ui: dashboard: add task summaries
>   ui: dashboard: make task summary time range configurable
> 
>  lib/pdm-api-types/src/lib.rs       |  46 +++++
>  server/src/api/remote_tasks.rs     |  88 +++++++-
>  server/src/remote_tasks/mod.rs     |  17 +-
>  ui/src/dashboard/filtered_tasks.rs | 297 +++++++++++++++++++++++++++
>  ui/src/dashboard/mod.rs            | 138 ++++++++++++-
>  ui/src/dashboard/tasks.rs          | 316 +++++++++++++++++++++++++++++
>  ui/src/remotes/tasks.rs            |  19 +-
>  ui/src/tasks.rs                    |  52 ++++-
>  ui/src/top_nav_bar.rs              |  18 +-
>  9 files changed, 946 insertions(+), 45 deletions(-)
>  create mode 100644 ui/src/dashboard/filtered_tasks.rs
>  create mode 100644 ui/src/dashboard/tasks.rs
> 



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