[pdm-devel] [PATCH datacenter-manager v2] server: api: resources: handle template in status count

Dominik Csapak d.csapak at proxmox.com
Fri Apr 18 08:15:16 CEST 2025


Originally, the PVE API schema description did not include the template
flag (so the information was not present in the rust types), and after
adding it, we forgot to update this part of the code to count the
templates correctly.

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
changes from v1:
* add commit message to explain why we do this now

 server/src/api/resources.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/src/api/resources.rs b/server/src/api/resources.rs
index 453d9e8..ddcee7e 100644
--- a/server/src/api/resources.rs
+++ b/server/src/api/resources.rs
@@ -191,14 +191,14 @@ pub async fn get_status(
                     _ => counts.storages.unknown += 1,
                 },
                 Resource::PveQemu(r) => match r.status.as_str() {
+                    _ if r.template => counts.qemu.template += 1,
                     "running" => counts.qemu.running += 1,
-                    // FIXME: handle templates
                     "stopped" => counts.qemu.stopped += 1,
                     _ => counts.qemu.unknown += 1,
                 },
                 Resource::PveLxc(r) => match r.status.as_str() {
+                    _ if r.template => counts.lxc.template += 1,
                     "running" => counts.lxc.running += 1,
-                    // FIXME: handle templates
                     "stopped" => counts.lxc.stopped += 1,
                     _ => counts.lxc.unknown += 1,
                 },
-- 
2.39.5





More information about the pdm-devel mailing list