Fix tracing startup: Jaeger via OTLP/gRPC + update tracing orbit deps#13708
Fix tracing startup: Jaeger via OTLP/gRPC + update tracing orbit deps#13708manodyaSenevirathne wants to merge 2 commits intowso2:masterfrom
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 16 minutes and 11 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates APIM tracing so the “jaeger” OpenTelemetry tracer exports spans via OTLP/gRPC (to address Jaeger startup/visibility issues) and refreshes related Jaeger/Zipkin orbit dependency versions.
Changes:
- Switch Jaeger telemetry exporter from
JaegerGrpcSpanExportertoOtlpGrpcSpanExporter. - Update the Jaeger telemetry default port to OTLP/gRPC default
4317. - Bump Jaeger client and Zipkin reporter/sender dependency versions in the root
pom.xml.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
pom.xml |
Bumps Jaeger/Zipkin versions (orbit + zipkin-reporter) to align with updated tracing stack. |
.../telemetry/TelemetryConstants.java |
Changes Jaeger default port from 14250 to 4317 to match OTLP/gRPC defaults. |
.../telemetry/JaegerTelemetry.java |
Replaces Jaeger gRPC exporter with OTLP/gRPC exporter for “jaeger” tracer. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Purpose
Resolves :
Changes
Orbit PR
Testing
Tested the setup mentioned in [1] , traces were visible in both jaeger and zipkin dashboards
[1] - https://apim.docs.wso2.com/en/latest/monitoring/observability/traces/monitoring-with-opentelemetry/