Add MeterRegistry param in callers

Author: nicktindall

server/src/main/java/org/elasticsearch/repositories/RepositoriesService.java

@@ -127,18 +127,6 @@ public class RepositoriesService extends AbstractLifecycleComponent implements C
     private final List<BiConsumer<Snapshot, IndexVersion>> preRestoreChecks;
     private final SnapshotMetrics snapshotMetrics;
 
-    public RepositoriesService(
-        Settings settings,
-        ClusterService clusterService,
-        Map<String, Repository.Factory> typesRegistry,
-        Map<String, Repository.Factory> internalTypesRegistry,
-        ThreadPool threadPool,
-        NodeClient client,
-        List<BiConsumer<Snapshot, IndexVersion>> preRestoreChecks
-    ) {
-        this(settings, clusterService, typesRegistry, internalTypesRegistry, threadPool, client, preRestoreChecks, MeterRegistry.NOOP);
-    }
-
     @SuppressWarnings("this-escape")
     public RepositoriesService(
         Settings settings,

server/src/test/java/org/elasticsearch/action/admin/cluster/repositories/reservedstate/ReservedRepositoryActionTests.java

@@ -22,6 +22,7 @@
 import org.elasticsearch.repositories.RepositoryException;
 import org.elasticsearch.repositories.RepositoryMissingException;
 import org.elasticsearch.reservedstate.TransformState;
+import org.elasticsearch.telemetry.metric.MeterRegistry;
 import org.elasticsearch.test.ESTestCase;
 import org.elasticsearch.threadpool.ThreadPool;
 import org.elasticsearch.xcontent.XContentParser;
@@ -143,7 +144,8 @@ public Repository create(ProjectId projectId, RepositoryMetadata metadata) {
                 Map.of("fs", fsFactory),
                 threadPool,
                 mock(NodeClient.class),
-                null
+                null,
+                MeterRegistry.NOOP
             )
         );
 

server/src/test/java/org/elasticsearch/action/admin/cluster/snapshots/status/TransportSnapshotsStatusActionTests.java

@@ -31,6 +31,7 @@
 import org.elasticsearch.tasks.CancellableTask;
 import org.elasticsearch.tasks.TaskCancelHelper;
 import org.elasticsearch.tasks.TaskCancelledException;
+import org.elasticsearch.telemetry.metric.MeterRegistry;
 import org.elasticsearch.test.ClusterServiceUtils;
 import org.elasticsearch.test.ESTestCase;
 import org.elasticsearch.test.transport.CapturingTransport;
@@ -74,7 +75,8 @@ public void initializeComponents() throws Exception {
             Map.of(),
             threadPool,
             nodeClient,
-            List.of()
+            List.of(),
+            MeterRegistry.NOOP
         );
         action = new TransportSnapshotsStatusAction(
             transportService,

server/src/test/java/org/elasticsearch/indices/cluster/IndicesClusterStateServiceRandomUpdatesTests.java

@@ -47,6 +47,7 @@
 import org.elasticsearch.indices.recovery.PeerRecoveryTargetService;
 import org.elasticsearch.indices.recovery.SnapshotFilesProvider;
 import org.elasticsearch.repositories.RepositoriesService;
+import org.elasticsearch.telemetry.metric.MeterRegistry;
 import org.elasticsearch.threadpool.TestThreadPool;
 import org.elasticsearch.threadpool.ThreadPool;
 import org.elasticsearch.transport.Transport;
@@ -549,7 +550,8 @@ private IndicesClusterStateService createIndicesClusterStateService(
             Collections.emptyMap(),
             threadPool,
             client,
-            List.of()
+            List.of(),
+            MeterRegistry.NOOP
         );
         final PeerRecoveryTargetService recoveryTargetService = new PeerRecoveryTargetService(
             client,

server/src/test/java/org/elasticsearch/repositories/RepositoriesServiceTests.java

@@ -53,6 +53,7 @@
 import org.elasticsearch.repositories.blobstore.MeteredBlobStoreRepository;
 import org.elasticsearch.snapshots.SnapshotId;
 import org.elasticsearch.snapshots.SnapshotInfo;
+import org.elasticsearch.telemetry.metric.MeterRegistry;
 import org.elasticsearch.test.ClusterServiceUtils;
 import org.elasticsearch.test.ESTestCase;
 import org.elasticsearch.threadpool.TestThreadPool;
@@ -155,7 +156,8 @@ public void setUp() throws Exception {
             typesRegistry,
             threadPool,
             client,
-            List.of()
+            List.of(),
+            MeterRegistry.NOOP
         );
 
         clusterService.start();

server/src/test/java/org/elasticsearch/snapshots/SnapshotResiliencyTests.java

@@ -187,6 +187,7 @@
 import org.elasticsearch.search.builder.SearchSourceBuilder;
 import org.elasticsearch.search.fetch.FetchPhase;
 import org.elasticsearch.telemetry.TelemetryProvider;
+import org.elasticsearch.telemetry.metric.MeterRegistry;
 import org.elasticsearch.telemetry.tracing.Tracer;
 import org.elasticsearch.test.ClusterServiceUtils;
 import org.elasticsearch.test.ESTestCase;
@@ -2410,7 +2411,8 @@ public RecyclerBytesStreamOutput newNetworkBytesStream() {
                     emptyMap(),
                     threadPool,
                     client,
-                    List.of()
+                    List.of(),
+                    MeterRegistry.NOOP
                 );
                 final ActionFilters actionFilters = new ActionFilters(emptySet());
                 snapshotsService = new SnapshotsService(