public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Lukas Wagner <l.wagner@proxmox.com>
To: pdm-devel@lists.proxmox.com
Subject: [PATCH datacenter-manager/proxmox-yew-comp v4 0/8] manual refresh button for remote task cache
Date: Fri,  6 Feb 2026 14:44:25 +0100	[thread overview]
Message-ID: <20260206134433.311230-1-l.wagner@proxmox.com> (raw)

NOTE: the PDM patches need to be applied on top of [1]

This adds the 
  POST /remotes/tasks/refresh[?remotes=...]

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 v3:
  - Rebase on top of the ParallelFetcher changes from [1]
  - Fix clippy issues (new patches)

Changes since v2:
  - Incorporate review feedback from Shannon:
    - Import cosmetics
    - Support refresh multiple remotes at once (renaming the 'remote' parameter to
      'remotes')
    - Improve docs for the API handler

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

[1]: https://lore.proxmox.com/all/20260206094304.117465-1-l.wagner@proxmox.com/T/#u


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 (6):
  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
  clippy: remote tasks: implement Default for TaskState
  clippy: remote tasks: remove unneeded enclosing Ok(...) and ?

 server/src/api/remote_tasks.rs                | 100 +++-
 .../tasks/remote_tasks.rs                     | 547 +----------------
 server/src/remote_tasks/mod.rs                |   1 +
 server/src/remote_tasks/refresh_task.rs       | 550 ++++++++++++++++++
 ui/src/remotes/tasks.rs                       |  14 +-
 5 files changed, 667 insertions(+), 545 deletions(-)
 create mode 100644 server/src/remote_tasks/refresh_task.rs


Summary over all repositories:
  7 files changed, 687 insertions(+), 547 deletions(-)

-- 
Generated by murpp 0.9.0




             reply	other threads:[~2026-02-06 13:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-06 13:44 Lukas Wagner [this message]
2026-02-06 13:44 ` [PATCH proxmox-yew-comp v4 1/2] task list: add support for starting refresh tasks Lukas Wagner
2026-02-06 13:44 ` [PATCH proxmox-yew-comp v4 2/2] loadable component: don't trigger reload when starting a task Lukas Wagner
2026-02-06 13:44 ` [PATCH datacenter-manager v4 1/6] remote tasks: move implementation to server crate Lukas Wagner
2026-02-06 13:44 ` [PATCH datacenter-manager v4 2/6] remote tasks: implement `refresh_task_cache` for manual task fetching Lukas Wagner
2026-02-06 13:44 ` [PATCH datacenter-manager v4 3/6] api: add /remotes/tasks/refresh Lukas Wagner
2026-02-06 13:44 ` [PATCH datacenter-manager v4 4/6] ui: remote task view: set refresh_task_url property for task viewer Lukas Wagner
2026-02-06 13:44 ` [PATCH datacenter-manager v4 5/6] clippy: remote tasks: implement Default for TaskState Lukas Wagner
2026-02-06 13:44 ` [PATCH datacenter-manager v4 6/6] clippy: remote tasks: remove unneeded enclosing Ok(...) and ? Lukas Wagner

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=20260206134433.311230-1-l.wagner@proxmox.com \
    --to=l.wagner@proxmox.com \
    --cc=pdm-devel@lists.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal