[PATCH datacenter-manager 02/15] task cache: fix missing cutoff state for PBS remotes

[Lukas Wagner <l.wagner@proxmox.com>]

The node success map, which is used to determine whether the cutoff in
the state file should be updated or not, uses 'localhost' as a nodename
for PBS remotes, since at the point where it is instantiated, we don't
know the real hostname of a PBS remote.

This means that we have to also use 'localhost' where we lookup the
status in the node success map, otherwise the state file will never
contain entries for PBS remotes, leading to the refresh task always
requesting a full task history.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
 server/src/remote_tasks/task_cache.rs | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/server/src/remote_tasks/task_cache.rs b/server/src/remote_tasks/task_cache.rs
index b0b4a837..b5fc3b21 100644
--- a/server/src/remote_tasks/task_cache.rs
+++ b/server/src/remote_tasks/task_cache.rs
@@ -15,7 +15,7 @@ use serde::{Deserialize, Serialize};
 
 use proxmox_sys::fs::CreateOptions;
 
-use pdm_api_types::RemoteUpid;
+use pdm_api_types::{RemoteUpid, remotes::RemoteType};
 
 /// Filename for the file containing running tasks.
 const ACTIVE_FILENAME: &str = "active";
@@ -424,7 +424,12 @@ impl WritableTaskCache {
                 }
             };
 
-            let node = native_upid.node();
+            let node = match task.upid.remote_type() {
+                RemoteType::Pve => native_upid.node(),
+                // The node success map uses 'localhost' as a node name for PBS remotes,
+                // not the one from the UPID.
+                RemoteType::Pbs => "localhost",
+            };
             let remote = task.upid.remote();
 
             if node_success_map.node_successful(remote, node) {
@@ -449,7 +454,12 @@ impl WritableTaskCache {
                 }
             };
 
-            let node = native_upid.node();
+            let node = match task.upid.remote_type() {
+                RemoteType::Pve => native_upid.node(),
+                // The node success map uses 'localhost' as a node name for PBS remotes,
+                // not the one from the UPID.
+                RemoteType::Pbs => "localhost",
+            };
             let remote = task.upid.remote();
 
             if node_success_map.node_successful(remote, node) {
-- 
2.47.3