Reinstate recovery planner plugin construction to the main construct …

…method (elastic#102206) It was moved inside the module by elastic#101123, this puts it back where it was
andreidan · Nov 15, 2023 · d44770d · d44770d
1 parent 6449899
commit d44770d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/server/src/main/java/org/elasticsearch/node/NodeConstruction.java b/server/src/main/java/org/elasticsearch/node/NodeConstruction.java
@@ -1016,7 +1016,7 @@ record PluginServiceInstances(
             compatibilityVersions,
             featureService
         );
-        this.nodeService = new NodeService(
+        nodeService = new NodeService(
             settings,
             threadPool,
             monitorService,
@@ -1102,8 +1102,8 @@ record PluginServiceInstances(
             loadDiagnosticServices(settings, discoveryModule.getCoordinator(), clusterService, transportService, featureService, threadPool)
         );
 
+        RecoveryPlannerService recoveryPlannerService = getRecoveryPlannerService(threadPool, clusterService, repositoryService);
         modules.add(b -> {
-            RecoveryPlannerService recoveryPlannerService = getRecoveryPlannerService(threadPool, clusterService, repositoryService);
             serviceProvider.processRecoverySettings(pluginsService, settingsModule.getClusterSettings(), recoverySettings);
             SnapshotFilesProvider snapshotFilesProvider = new SnapshotFilesProvider(repositoryService);
             var peerRecovery = new PeerRecoverySourceService(