fix: use jaeger-exporter

biothings · Dec 15, 2023 · 3e83170 · 3e83170
1 parent 7cd447a
commit 3e83170
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/package.json b/package.json
@@ -68,6 +68,7 @@
     "@bull-board/express": "^5.9.1",
     "@opentelemetry/api": "^1.7.0",
     "@opentelemetry/auto-instrumentations-node": "^0.40.2",
+    "@opentelemetry/exporter-jaeger": "^1.19.0",
     "@opentelemetry/exporter-metrics-otlp-proto": "^0.45.1",
     "@opentelemetry/exporter-trace-otlp-proto": "^0.45.1",
     "@opentelemetry/resources": "^1.18.1",

diff --git a/src/controllers/opentelemetry.js b/src/controllers/opentelemetry.js
@@ -1,17 +1,15 @@
 const opentelemetry = require("@opentelemetry/sdk-node");
 const { getNodeAutoInstrumentations } = require("@opentelemetry/auto-instrumentations-node");
-const { OTLPTraceExporter } = require("@opentelemetry/exporter-trace-otlp-proto");
-const { OTLPMetricExporter } = require("@opentelemetry/exporter-metrics-otlp-proto");
-const { PeriodicExportingMetricReader, ConsoleMetricExporter } = require("@opentelemetry/sdk-metrics");
 const { Resource } = require("@opentelemetry/resources");
-const { ConsoleSpanExporter } = require("@opentelemetry/sdk-trace-node");
 const Debug = require("debug");
 const debug = Debug("bte:biothings-explorer:otel-init");
+const { JaegerExporter } = require("@opentelemetry/exporter-jaeger");
 
 debug("Initializing Opentelemetry instrumentation...");
 const sdk = new opentelemetry.NodeSDK({
-  traceExporter: new OTLPTraceExporter({
-    url: (process.env.JAEGER_URL ?? "http://localhost:4318") + "/v1/traces",
+  traceExporter: new JaegerExporter({
+    host: process.env.JAEGER_HOST ?? "jaeger-otel-agent.sri",
+    port: parseInt(process.env.JAEGER_PORT ?? "6381"),
   }),
   instrumentations: [getNodeAutoInstrumentations()],
   resource: new Resource({