Fix domain filtering for multi-cluster HTTPRoutes

Store per-cluster domain filters in HTTPRoute annotations and use them
when generating Homer config items. This ensures that HTTPRoutes with
multiple hostnames only show the hostnames matching the cluster's domain
filter.

Changes:
- Store domain filters as annotation on HTTPRoutes in cluster_manager
- Read domain-filters annotation in Homer config generator
- Remove duplicate domain filtering in dashboard controller
- Pass nil domain filters since filtering is done per-HTTPRoute

Author: rajsinghtech

internal/controller/cluster_manager.go

@@ -21,6 +21,7 @@ import (
 	"crypto/sha256"
 	"encoding/hex"
 	"fmt"
+	"strings"
 	"sync"
 	"time"
 
@@ -523,6 +524,11 @@ func (m *ClusterManager) discoverClusterHTTPRoutes(ctx context.Context, cluster
 			}
 			clusterHTTPRoutes.Items[i].Annotations["homer.rajsingh.info/cluster"] = cluster.Name
 
+			// Store domain filters as annotation so Homer config generator knows which hostnames to show
+			if len(domainFilters) > 0 {
+				clusterHTTPRoutes.Items[i].Annotations["homer.rajsingh.info/domain-filters"] = strings.Join(domainFilters, ",")
+			}
+
 			filtered = append(filtered, clusterHTTPRoutes.Items[i])
 		}
 	}

internal/controller/dashboard_controller.go

@@ -872,17 +872,13 @@ func (r *DashboardReconciler) createConfigMap(ctx context.Context, homerConfig *
 				// Continue with partial results
 			}
 
-			// Aggregate all discovered HTTPRoutes
+			// Aggregate all discovered HTTPRoutes (already filtered by ClusterManager)
 			for clusterName, httproutes := range clusterHTTPRoutes {
 				log := log.FromContext(ctx)
 				log.V(1).Info("Discovered HTTPRoutes from cluster", "cluster", clusterName, "count", len(httproutes))
 
-				// Apply domain filters (already filtered by ClusterManager but apply domain filters)
-				for _, httproute := range httproutes {
-					if homer.MatchesDomainFilters(r.getHTTPRouteHosts(&httproute), dashboard.Spec.DomainFilters) {
-						filteredHTTPRoutes = append(filteredHTTPRoutes, httproute)
-					}
-				}
+				// HTTPRoutes are already filtered by ClusterManager with per-cluster domain filters
+				filteredHTTPRoutes = append(filteredHTTPRoutes, httproutes...)
 			}
 		} else {
 			// Fall back to single-cluster discovery
@@ -902,7 +898,9 @@ func (r *DashboardReconciler) createConfigMap(ctx context.Context, homerConfig *
 			}
 		}
 
-		return homer.CreateConfigMapWithHTTPRoutes(homerConfig, dashboard.Name, dashboard.Namespace, filteredIngressList, filteredHTTPRoutes, dashboard, dashboard.Spec.DomainFilters)
+		// Don't pass domain filters since HTTPRoutes are already filtered by ClusterManager
+		// and we want to show all hostnames from each HTTPRoute
+		return homer.CreateConfigMapWithHTTPRoutes(homerConfig, dashboard.Name, dashboard.Namespace, filteredIngressList, filteredHTTPRoutes, dashboard, nil)
 	}
 
 	return homer.CreateConfigMap(homerConfig, dashboard.Name, dashboard.Namespace, filteredIngressList, dashboard)

pkg/homer/config.go

@@ -1250,11 +1250,21 @@ func updateHomerConfigWithHTTPRoutes(
 		// This allows for cleanup when all hostnames are removed
 		return
 	} else {
+		// Check if HTTPRoute has per-cluster domain filters annotation
+		effectiveDomainFilters := domainFilters
+		if filterAnnotation, ok := httproute.ObjectMeta.Annotations["homer.rajsingh.info/domain-filters"]; ok && filterAnnotation != "" {
+			// Use per-cluster domain filters from annotation
+			effectiveDomainFilters = strings.Split(filterAnnotation, ",")
+			for i := range effectiveDomainFilters {
+				effectiveDomainFilters[i] = strings.TrimSpace(effectiveDomainFilters[i])
+			}
+		}
+
 		// Create separate item for each hostname that matches domain filters
 		var filteredHostnames []gatewayv1.Hostname
 		for _, hostname := range httproute.Spec.Hostnames {
 			hostStr := string(hostname)
-			if utils.MatchesHostDomainFilters(hostStr, domainFilters) {
+			if utils.MatchesHostDomainFilters(hostStr, effectiveDomainFilters) {
 				filteredHostnames = append(filteredHostnames, hostname)
 			}
 		}