Re: [pdm-devel] [PATCH datacenter-manager/proxmox-yew-comp v2 0/6] manual refresh button for remote task cache

["Shannon Sterz" <s.sterz@proxmox.com>]

On Thu Dec 18, 2025 at 3:20 PM CET, Lukas Wagner wrote:
> This adds the
>   POST /remotes/tasks/refresh[?remote=...]
>
> endpoint for manually triggering task collection from a single or all remotes.
>
> The task viewer component is changed so that the refresh button can be wired
> up to trigger this new endpoint.
>
> Changes since v1:
>   - Drop the patch which adds the task descriptions of the new
>   refresh-task - this one is now included in another patch series
>   - Fix a bug where the view would not automatically display new tasks
>   after the refresh task finished
>
>
> proxmox-yew-comp:
>
> Lukas Wagner (2):
>   task list: add support for starting refresh tasks
>   loadable component: don't trigger reload when starting a task
>
>  src/loadable_component.rs |  1 -
>  src/tasks.rs              | 21 ++++++++++++++++++++-
>  2 files changed, 20 insertions(+), 2 deletions(-)
>
>
> proxmox-datacenter-manager:
>
> Lukas Wagner (4):
>   remote tasks: move implementation to server crate
>   remote tasks: implement `refresh_task_cache` for manual task fetching
>   api: add /remotes/tasks/refresh
>   ui: remote task view: set refresh_task_url property for task viewer
>
>  server/src/api/remote_tasks.rs                |  90 ++-
>  .../tasks/remote_tasks.rs                     | 535 +----------------
>  server/src/remote_tasks/mod.rs                |   1 +
>  server/src/remote_tasks/refresh_task.rs       | 544 ++++++++++++++++++
>  ui/src/remotes/tasks.rs                       |  14 +-
>  5 files changed, 650 insertions(+), 534 deletions(-)
>  create mode 100644 server/src/remote_tasks/refresh_task.rs
>
>
> Summary over all repositories:
>   7 files changed, 670 insertions(+), 536 deletions(-)

took a look at this series, generally the changes look good to me, just
had two smaller notes that i left on the respective patches. so consider
this:

Reviewed-by: Shannon Sterz <s.sterz@proxmox.com>


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