Add tracer to getTransports (opensearch-project#556)

Signed-off-by: Gagan Juneja <gjjuneja@amazon.com> Co-authored-by: Gagan Juneja <gjjuneja@amazon.com>
shivamg2017 · Oct 5, 2023 · 999c689 · 999c689
1 parent e7713ba
commit 999c689
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/main/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerPlugin.java b/src/main/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerPlugin.java
@@ -92,6 +92,7 @@
 import org.opensearch.repositories.RepositoriesService;
 import org.opensearch.rest.RestController;
 import org.opensearch.script.ScriptService;
+import org.opensearch.telemetry.tracing.Tracer;
 import org.opensearch.threadpool.ThreadPool;
 import org.opensearch.transport.Transport;
 import org.opensearch.transport.TransportInterceptor;
@@ -369,7 +370,8 @@ public Map<String, Supplier<Transport>> getTransports(
             PageCacheRecycler pageCacheRecycler,
             CircuitBreakerService circuitBreakerService,
             NamedWriteableRegistry namedWriteableRegistry,
-            NetworkService networkService) {
+            NetworkService networkService,
+            Tracer tracer) {
         OpenSearchResources.INSTANCE.setSettings(settings);
         OpenSearchResources.INSTANCE.setCircuitBreakerService(circuitBreakerService);
         return Collections.emptyMap();

diff --git a/src/test/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerPluginTests.java b/src/test/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerPluginTests.java
@@ -43,6 +43,7 @@
 import org.opensearch.plugins.ActionPlugin.ActionHandler;
 import org.opensearch.rest.RestController;
 import org.opensearch.rest.RestHandler;
+import org.opensearch.telemetry.tracing.noop.NoopTracer;
 import org.opensearch.test.OpenSearchTestCase;
 import org.opensearch.threadpool.TestThreadPool;
 import org.opensearch.threadpool.ThreadPool;
@@ -154,7 +155,14 @@ public void testCreateComponents() {
     @Test
     public void testGetTransports() {
         Map<String, Supplier<Transport>> map =
-                plugin.getTransports(settings, threadPool, null, circuitBreakerService, null, null);
+                plugin.getTransports(
+                        settings,
+                        threadPool,
+                        null,
+                        circuitBreakerService,
+                        null,
+                        null,
+                        NoopTracer.INSTANCE);
         assertEquals(0, map.size());
         assertEquals(settings, OpenSearchResources.INSTANCE.getSettings());
         assertEquals(