-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
removing jaeger exporters #890
Conversation
Jaeger has dropped support for the jaeger exporter in favour of opentelemetry exporter. The upstream repos are now archived. Support has been dropped from the opentelemetry collector, and other SIGs should follow soon. ref: open-telemetry/opentelemetry-specification#2858 and open-telemetry/opentelemetry-specification#2859
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #890 +/- ##
============================================
- Coverage 81.59% 81.11% -0.49%
+ Complexity 2010 1905 -105
============================================
Files 264 252 -12
Lines 5226 4929 -297
============================================
- Hits 4264 3998 -266
+ Misses 962 931 -31
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@brettmc Is not that too early? It seems that there will be deprecation period regarding removing it from spec |
I missed SIG yesterday, but Bob noted that he thought better to remove it before beta. It's definitely going away (upstream are not even accepting security fixes anymore), so removing it earlier means less for us to maintain. |
Bob mentioned that yesterday, but after reading attached tickets, I was not sure we should do that now, however I agree that it should not apply to us |
Jaeger has dropped support for the jaeger exporter in favour of opentelemetry exporter. The upstream repos are now archived. Support has been dropped from the opentelemetry collector, and other SIGs should follow soon.
ref: open-telemetry/opentelemetry-specification#2858 and open-telemetry/opentelemetry-specification#2859