SPAM: [PATCH datacenter-manager 0/2] Remove running tasks from

SPAM: [PATCH datacenter-manager 0/2] Remove running tasks from

[Jonas Theisen]

This patch series fixes issues with filtering for errored tasks
where running tasks would get wrongly displayed or returned through
the API.

The first patch is analogous to the patch suggested for PBS in [1]
and fixes the PDM local tasks returned by the API when queried for
errors.

The second patch was added to align the behavior of tasks imported
from PVE to the tasks from PBS or returned by PDM.
This indirectly also fixes the issue that running PVE tasks get
displayed in the Remote tab of PDM when filtering for errors
while running PBS tasks got correctly filtered out.

[1] https://lore.proxmox.com/pbs-devel/20260827154740.427154-1-j.theisen@proxmox.com/

Jonas Theisen (2):
  api: Remove running tasks from errors output
  remote-tasks: Align fetched PVE tasks to PBS and PDM API

 server/src/api/nodes/tasks.rs           | 1 +
 server/src/remote_tasks/refresh_task.rs | 8 +++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

-- 
2.47.3

[PATCH datacenter-manager 1/2] api: Remove running tasks from errors output

[Jonas Theisen]

Currently running tasks are returned by the PDM API in the query
for errored tasks.
This patch adds an additional condition to filter out running tasks
when queried for errored tasks

This is related to the patch suggested for the PBS API at
https://lore.proxmox.com/pbs-devel/20260827154740.427154-1-j.theisen@proxmox.com/

Signed-off-by: Jonas Theisen <j.theisen@proxmox.com>
---
 server/src/api/nodes/tasks.rs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/server/src/api/nodes/tasks.rs b/server/src/api/nodes/tasks.rs
index 31ddb8f1..1cca3c0a 100644
--- a/server/src/api/nodes/tasks.rs
+++ b/server/src/api/nodes/tasks.rs
@@ -155,6 +155,7 @@ pub fn list_tasks(
                     continue;
                 }
             }
+            (None, _) if errors => continue,
             (None, Some(_)) => continue,
             _ => {}
         }
-- 
2.47.3

[PATCH datacenter-manager 2/2] remote-tasks: Align fetched PVE tasks to PBS and PDM API

[Jonas Theisen]

Running tasks fetched from PVE's API have the status "RUNNING"
while PBS and PDM have None.
Since the filtering logic in PDM also expects the state to be None
for running tasks this patch aligns the status of fetched tasks.

With this applied running PVE tasks no longer display when filtering
for errored tasks in the Remote tab of PDM.

Signed-off-by: Jonas Theisen <j.theisen@proxmox.com>
---
 server/src/remote_tasks/refresh_task.rs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/server/src/remote_tasks/refresh_task.rs b/server/src/remote_tasks/refresh_task.rs
index 668b63e4..3275bdb8 100644
--- a/server/src/remote_tasks/refresh_task.rs
+++ b/server/src/remote_tasks/refresh_task.rs
@@ -504,11 +504,17 @@ async fn poll_single_tracked_task(remote: Remote, task: RemoteUpid) -> (RemoteUp
 fn map_pve_task(task: pve_api_types::ListTasksResponse, remote: String) -> TaskCacheItem {
     let remote_upid = RemoteUpid::new(remote, RemoteType::Pve, task.upid);
 
+    // This is to align the PVE tasks to PBS and PDM tasks
+    let status = match task.status {
+        Some(ref s) if s == "RUNNING" => None,
+        other => other,
+    };
+
     TaskCacheItem {
         upid: remote_upid,
         starttime: task.starttime,
         endtime: task.endtime,
-        status: task.status,
+        status: status,
     }
 }
 
-- 
2.47.3