From 161bbf5ec4fe1cb79cb9d13d1b4da10034dc8300 Mon Sep 17 00:00:00 2001
From: Pulse Monitor <pulse@pulse-monitor.dev>
Date: Mon, 11 Aug 2025 10:13:12 +0000
Subject: [PATCH] fix: exclude ct templates and isos from backup tab (fixes
 #265)

- filter out vztmpl (container templates) from backup list
- filter out iso files from backup list
- only show actual vm/container backups in the backup tab
- remove unnecessary checks for template/iso content types
---
 internal/monitoring/monitor.go | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/internal/monitoring/monitor.go b/internal/monitoring/monitor.go
index 7df74bfc5..4d6924883 100644
--- a/internal/monitoring/monitor.go
+++ b/internal/monitoring/monitor.go
@@ -1526,8 +1526,8 @@ func (m *Monitor) pollStorageBackups(ctx context.Context, instanceName string, c
 
 		// For each storage that can contain backups or templates
 		for _, storage := range storages {
-			// Check if storage supports backup or template content
-			if !strings.Contains(storage.Content, "backup") && !strings.Contains(storage.Content, "vztmpl") && !strings.Contains(storage.Content, "iso") {
+			// Check if storage supports backup content
+			if !strings.Contains(storage.Content, "backup") {
 				continue
 			}
 
@@ -1550,13 +1550,14 @@ func (m *Monitor) pollStorageBackups(ctx context.Context, instanceName string, c
 				}
 				seenVolids[content.Volid] = true
 
+				// Skip templates and ISOs - they're not backups
+				if content.Content == "vztmpl" || content.Content == "iso" {
+					continue
+				}
+
 				// Determine type from content type and volid
 				backupType := "unknown"
-				if content.Content == "vztmpl" {
-					backupType = "vztmpl"
-				} else if content.Content == "iso" {
-					backupType = "iso"
-				} else if strings.Contains(content.Volid, "/vm/") || strings.Contains(content.Volid, "qemu") {
+				if strings.Contains(content.Volid, "/vm/") || strings.Contains(content.Volid, "qemu") {
 					backupType = "qemu"
 				} else if strings.Contains(content.Volid, "/ct/") || strings.Contains(content.Volid, "lxc") {
 					backupType = "lxc"