show all route names in endpoint name

reconcile/endpoints_discovery/integration.py

@@ -137,21 +137,27 @@ def get_routes(self, oc_map: OCMap, namespace: NamespaceV1) -> list[Route]:
             )
             return []
 
-        routes = {}
+        routes: dict[str, list[Route]] = {}
         for item in sorted(
             oc.get_items(kind="Route", namespace=namespace.name),
             key=lambda x: x["metadata"]["name"],
-            reverse=True,
         ):
             tls = bool(item["spec"].get("tls"))
             host = item["spec"]["host"]
-            routes[f"{host}:{tls}"] = Route(
-                name=item["metadata"]["name"],
-                host=host,
-                tls=tls,
+            # group all routes with the same hostname/tls
+            routes.setdefault(f"{host}:{tls}", []).append(
+                Route(
+                    name=item["metadata"]["name"],
+                    host=host,
+                    tls=tls,
+                )
             )
 
-        return list(routes.values())
+        return [
+            # merge all routes with the same hostname into one and combine the names
+            Route(name="|".join(r.name for r in rs), host=rs[0].host, tls=rs[0].tls)
+            for rs in routes.values()
+        ]
 
     def get_endpoint_changes(
         self,

reconcile/test/endpoints_discovery/test_endpoints_discovery_integration.py

@@ -88,7 +88,7 @@ def test_endpoints_discovery_integration_get_routes(
     # see fake_route fixture!
     assert len(routes) == 1
     assert isinstance(routes[0], Route)
-    assert routes[0].name == "fake-route"
+    assert routes[0].name == "fake-route|zzz-fake-route"
 
 
 def test_endpoints_discovery_integration_get_endpoint_changes_no_routes_no_endpoints(
@@ -158,9 +158,9 @@ def test_endpoints_discovery_integration_get_apps(
             path="/path/app-1.yml",
             endpoints_to_add=[
                 Endpoint(
-                    name="endpoints-discovery/cluster-1/app-1-ns-1/fake-route",
+                    name="endpoints-discovery/cluster-1/app-1-ns-1/fake-route|zzz-fake-route",
                     data={
-                        "name": "endpoints-discovery/cluster-1/app-1-ns-1/fake-route",
+                        "name": "endpoints-discovery/cluster-1/app-1-ns-1/fake-route|zzz-fake-route",
                         "url": "https://fake-route.com:80",
                     },
                 )
@@ -173,9 +173,9 @@ def test_endpoints_discovery_integration_get_apps(
             path="/path/app-2.yml",
             endpoints_to_add=[
                 Endpoint(
-                    name="endpoints-discovery/cluster-1/app-2-ns-1/fake-route",
+                    name="endpoints-discovery/cluster-1/app-2-ns-1/fake-route|zzz-fake-route",
                     data={
-                        "name": "endpoints-discovery/cluster-1/app-2-ns-1/fake-route",
+                        "name": "endpoints-discovery/cluster-1/app-2-ns-1/fake-route|zzz-fake-route",
                         "url": "https://fake-route.com:80",
                     },
                 )