Treat all backends as alive in Noop monitor

Author: oneonestar

gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/NoopClusterStatsMonitor.java

@@ -21,6 +21,11 @@ public class NoopClusterStatsMonitor
     @Override
     public ClusterStats monitor(ProxyBackendConfiguration backend)
     {
-        return ClusterStats.builder(backend.getName()).build();
+        return ClusterStats.builder(backend.getName())
+                .trinoStatus(TrinoStatus.HEALTHY)
+                .proxyTo(backend.getProxyTo())
+                .externalUrl(backend.getExternalUrl())
+                .routingGroup(backend.getRoutingGroup())
+                .build();
     }
 }

gateway-ha/src/test/java/io/trino/gateway/ha/clustermonitor/TestClusterStatsMonitor.java

@@ -79,6 +79,19 @@ void testJmxMonitor()
         testClusterStatsMonitor(backendStateConfiguration -> new ClusterStatsJmxMonitor(new JettyHttpClient(new HttpClientConfig()), backendStateConfiguration));
     }
 
+    @Test
+    void testNoopMonitor()
+    {
+        ClusterStatsMonitor monitor = new NoopClusterStatsMonitor();
+        ProxyBackendConfiguration proxyBackend = new ProxyBackendConfiguration();
+        proxyBackend.setProxyTo("http://localhost:61234");  // assume no service running on this port
+        proxyBackend.setName("test_cluster");
+
+        ClusterStats stats = monitor.monitor(proxyBackend);
+        assertThat(stats.clusterId()).isEqualTo("test_cluster");
+        assertThat(stats.trinoStatus()).isEqualTo(TrinoStatus.HEALTHY);
+    }
+
     @Test
     void testJmxMonitorWithBadRequest()
     {