Load ActiveClusterMonitor by default

Author: oneonestar

docs/quickstart-config.yaml

@@ -12,6 +12,3 @@ dataStore:
 
 clusterStatsConfiguration:
   monitorType: INFO_API
-
-managedApps:
-  - io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor

gateway-ha/gateway-ha-config-docker.yml

@@ -15,6 +15,3 @@ dataStore:
 
 clusterStatsConfiguration:
   monitorType: INFO_API
-
-managedApps:
-  - io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor

gateway-ha/gateway-ha-config.yml

@@ -19,6 +19,3 @@ clusterStatsConfiguration:
 # This can be adjusted based on the coordinator state
 monitor:
   taskDelaySeconds: 10
-
-managedApps:
-  - io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor

gateway-ha/src/main/java/io/trino/gateway/baseapp/BaseApp.java

@@ -17,6 +17,7 @@
 import com.google.inject.Module;
 import com.google.inject.Scopes;
 import io.airlift.log.Logger;
+import io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor;
 import io.trino.gateway.ha.clustermonitor.ForMonitor;
 import io.trino.gateway.ha.config.HaGatewayConfiguration;
 import io.trino.gateway.ha.handler.ProxyHandlerStats;
@@ -118,6 +119,7 @@ public static List<Module> addModules(HaGatewayConfiguration configuration)
     public void configure(Binder binder)
     {
         binder.bind(HaGatewayConfiguration.class).toInstance(configuration);
+        binder.bind(ActiveClusterMonitor.class).in(Scopes.SINGLETON);
         registerAuthFilters(binder);
         registerResources(binder);
         registerProxyResources(binder);
@@ -131,7 +133,7 @@ public void configure(Binder binder)
     private static void addManagedApps(HaGatewayConfiguration configuration, Binder binder)
     {
         if (configuration.getManagedApps() == null) {
-            logger.error("No managed apps found");
+            logger.info("No managed apps found");
             return;
         }
         configuration.getManagedApps().forEach(

gateway-ha/src/test/resources/test-config-template.yml

@@ -10,9 +10,6 @@ dataStore:
   driver: org.h2.Driver
   runMigrationsEnabled: false
 
-managedApps:
-  - io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor
-
 clusterStatsConfiguration:
   monitorType: INFO_API
 

gateway-ha/src/test/resources/test-config-with-routing-template.yml

@@ -9,9 +9,6 @@ dataStore:
   driver: org.h2.Driver
   runMigrationsEnabled: false
 
-managedApps:
-  - io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor
-
 clusterStatsConfiguration:
   monitorType: INFO_API
 

gateway-ha/src/test/resources/test-config-without-x-forwarded-template.yml

@@ -9,9 +9,6 @@ dataStore:
   driver: org.h2.Driver
   runMigrationsEnabled: false
 
-managedApps:
-  - io.trino.gateway.ha.clustermonitor.ActiveClusterMonitor
-
 clusterStatsConfiguration:
   monitorType: INFO_API