perform platform enablement check earlier

cryostatio · Feb 6, 2024 · f22bfb7 · f22bfb7
1 parent 05d7538
commit f22bfb7
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/main/java/io/cryostat/discovery/BuiltInDiscovery.java b/src/main/java/io/cryostat/discovery/BuiltInDiscovery.java
@@ -74,6 +74,12 @@ public void start(Promise<Void> start) {
                 .distinct()
                 .forEach(
                         platform -> {
+                            if (!platform.isEnabled()) {
+                                logger.info(
+                                        "Skipping built-in discovery with {} : not enabled",
+                                        platform.getClass().getSimpleName());
+                                return;
+                            }
                             logger.info(
                                     "Starting built-in discovery with {}",
                                     platform.getClass().getSimpleName());
@@ -109,11 +115,9 @@ public void start(Promise<Void> start) {
                                     .onSuccess(
                                             subtree -> storage.update(id, subtree.getChildren()));
                             try {
-                                if (platform.isEnabled()) {
-                                    platform.start();
-                                    platform.load(promise);
-                                    enabledClients.add(platform);
-                                }
+                                platform.start();
+                                platform.load(promise);
+                                enabledClients.add(platform);
                             } catch (Exception e) {
                                 logger.warn(e);
                             }