BUG/MINOR: prevents clash on custom routes names to avoid unecessary …

…restarts
haproxytech · Sep 2, 2023 · ecd6505 · ecd6505
1 parent 6d18884
commit ecd6505
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/pkg/ingress/ingress.go b/pkg/ingress/ingress.go
@@ -15,6 +15,7 @@
 package ingress
 
 import (
+	"fmt"
 	"path/filepath"
 
 	"github.com/haproxytech/kubernetes-ingress/pkg/annotations"
@@ -112,8 +113,9 @@ func (i *Ingress) handlePath(k store.K8s, h haproxy.HAProxy, host string, path *
 
 	routeACLAnn := a.String("route-acl", svc.GetResource().Annotations)
 	if routeACLAnn == "" {
-		if _, ok := route.CustomRoutes[backendName]; ok {
-			delete(route.CustomRoutes, backendName)
+		customRouteID := fmt.Sprintf("%s-%s", host, backendName)
+		if _, ok := route.CustomRoutes[customRouteID]; ok {
+			delete(route.CustomRoutes, customRouteID)
 			logger.Debugf("Custom Route to backend '%s' deleted, reload required", backendName)
 			routeReload = true
 		}

diff --git a/pkg/route/route.go b/pkg/route/route.go
@@ -125,8 +125,9 @@ func AddCustomRoute(route Route, routeACLAnn string, api api.HAProxyClient) (rel
 			return
 		}
 	}
-	if acl := CustomRoutes[route.BackendName]; acl != routeCond {
-		CustomRoutes[route.BackendName] = routeCond
+	customRouteID := fmt.Sprintf("%s-%s", route.Host, route.BackendName)
+	if acl := CustomRoutes[customRouteID]; acl != routeCond {
+		CustomRoutes[customRouteID] = routeCond
 		reload = true
 		logger.Debugf("Custom Route to backend '%s' added, reload required", route.BackendName)
 	}