migrated to opentelemetry instead of opentracing for tracing support

automatiko-io · Sep 21, 2023 · 2a70c9b · 2a70c9b
1 parent 4f0b7c2
commit 2a70c9b
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 7 deletions.
diff --git a/weather-conditions/README.md b/weather-conditions/README.md
@@ -10,10 +10,10 @@ See complete description of this example [here](https://docs.automatiko.io/main/
 ## Run it
 
 The only thing that is needed is Jaeger (for observability/opentracing) running that will be accessible. One can be 
-started with
+started with is configured in docker compose file in this example, just execute:
 
 ````
-docker run -p 5775:5775/udp -p 6831:6831/udp -p 6832:6832/udp -p 5778:5778 -p 16686:16686 -p 14268:14268 jaegertracing/all-in-one:latest
+docker-compose up -d
 ````
 
 NOTE: To run this service API keys for [IPSTACK](https://ipstack.com/) and [Open Weather API](https://openweathermap.org/) is required. These can be freely obtained from the websites.

diff --git a/weather-conditions/docker-compose.yaml b/weather-conditions/docker-compose.yaml
@@ -0,0 +1,14 @@
+version: "2"
+services:
+
+  # Jaeger
+  jaeger-all-in-one:
+    image: jaegertracing/all-in-one:latest
+    ports:
+      - "16686:16686" # Jaeger UI
+      - "14268:14268" # Receive legacy OpenTracing traces, optional
+      - "4317:4317"   # OTLP gRPC receiver
+      - "4318:4318"   # OTLP HTTP receiver, not yet used by Quarkus, optional
+      - "14250:14250" # Receive from external otel-collector, optional
+    environment:
+      - COLLECTOR_OTLP_ENABLED=true
diff --git a/weather-conditions/pom.xml b/weather-conditions/pom.xml
@@ -43,7 +43,7 @@
 
     <dependency>
       <groupId>io.quarkus</groupId>
-      <artifactId>quarkus-smallrye-opentracing</artifactId>
+      <artifactId>quarkus-opentelemetry</artifactId>
     </dependency>
 
     <dependency>

diff --git a/weather-conditions/src/main/resources/application.properties b/weather-conditions/src/main/resources/application.properties
@@ -13,8 +13,6 @@ quarkus.log.category."org.openapitools.codegen".level=OFF
 
 quarkus.jib.base-jvm-image=eclipse-temurin:17-jre-alpine
 
-quarkus.jaeger.service-name=automatiko-weather-condition-service
-quarkus.jaeger.sampler-type=const
-quarkus.jaeger.sampler-param=1
+#quarkus.otel.exporter.otlp.traces.endpoint=http://localhost:4317
 
-%test.quarkus.jaeger.enable=false
+%test.quarkus.otel.sdk.disabled=true