[PATCH datacenter-manager 4/4] fix #7135: ui: correct calculations for shared storages

[Lukas Sichert <l.sichert@proxmox.com>]

Currently the storage calculation for a cluster in the
datacenter-manager is incorrect when a shared storage is involved. Each
node reports the shared storage separately, causing it to be counted
multiple times in the total capacity.

To be able to handle shared storages correctly, introduce a `shared`
field to PveStorageResource and parse it from the API-Resonse. After
that use the field and the storage name to only count the shared storage
once. This also aligns pdm storage calculation with the one in the
pve-frontend.

Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
---
 cli/client/src/resources.rs       | 1 +
 lib/pdm-api-types/src/resource.rs | 2 ++
 server/src/api/resources.rs       | 1 +
 server/src/views/tests.rs         | 1 +
 ui/src/pve/remote_overview.rs     | 9 +++++++--
 5 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/cli/client/src/resources.rs b/cli/client/src/resources.rs
index 9b91a4b..74763bf 100644
--- a/cli/client/src/resources.rs
+++ b/cli/client/src/resources.rs
@@ -85,6 +85,7 @@ impl fmt::Display for PrintResource<resource::PveStorageResource> {
             disk,
             maxdisk,
             id: _,
+            shared: _,
             ref storage,
             ref node,
             ref status,
diff --git a/lib/pdm-api-types/src/resource.rs b/lib/pdm-api-types/src/resource.rs
index 93f8bd2..b68777c 100644
--- a/lib/pdm-api-types/src/resource.rs
+++ b/lib/pdm-api-types/src/resource.rs
@@ -355,6 +355,8 @@ pub struct PveStorageResource {
     pub node: String,
     /// Storage status
     pub status: String,
+    /// shared flag
+    pub shared: bool,
 }
 
 #[api]
diff --git a/server/src/api/resources.rs b/server/src/api/resources.rs
index 5d5fe2c..a966109 100644
--- a/server/src/api/resources.rs
+++ b/server/src/api/resources.rs
@@ -1135,6 +1135,7 @@ pub(super) fn map_pve_storage(
             storage: resource.storage.unwrap_or_default(),
             node: resource.node.unwrap_or_default(),
             status: resource.status.unwrap_or_default(),
+            shared: resource.shared.unwrap_or_default(),
         }),
         _ => None,
     }
diff --git a/server/src/views/tests.rs b/server/src/views/tests.rs
index 9f49620..b8c63b4 100644
--- a/server/src/views/tests.rs
+++ b/server/src/views/tests.rs
@@ -14,6 +14,7 @@ fn make_storage_resource(remote: &str, node: &str, storage_name: &str) -> Resour
         storage: storage_name.into(),
         node: node.into(),
         status: "available".into(),
+        shared: false,
     })
 }
 
diff --git a/ui/src/pve/remote_overview.rs b/ui/src/pve/remote_overview.rs
index e452131..5b63539 100644
--- a/ui/src/pve/remote_overview.rs
+++ b/ui/src/pve/remote_overview.rs
@@ -1,3 +1,4 @@
+use std::collections::HashSet;
 use std::rc::Rc;
 
 use proxmox_yew_comp::{Status, StatusRow};
@@ -72,12 +73,16 @@ impl RemotePanelComp {
         let mut nodes = 0;
         let mut cpu_usage = 0.0;
         let mut level = None;
+        let mut seen_shared_storages: HashSet<String> = HashSet::new();
 
         for res in ctx.props().resources.iter() {
             match res {
                 PveResource::Storage(store) => {
-                    storage += store.disk;
-                    max_storage += store.maxdisk;
+                    let storage_name = store.storage.clone();
+                    if seen_shared_storages.insert(storage_name) {
+                        storage += store.disk;
+                        max_storage += store.maxdisk;
+                    }
                 }
                 PveResource::Qemu(qemu) => {
                     guests += 1;
-- 
2.47.3