-
Notifications
You must be signed in to change notification settings - Fork 871
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
Update the OpenTelemetry SDK version to 1.35.0 and remove jaeger exporter #10524
Update the OpenTelemetry SDK version to 1.35.0 and remove jaeger exporter #10524
Conversation
is removing jaeger exporter necessary? would be nice not to do that in a patch release if we don't have to |
Jaeger exporter was removed from sdk. That leaves us the following options
|
I think my preference would be to not update the SDK in that case. @jack-berg @jkwatson any thoughts? thanks |
No opinion from me. I don't use the agent, and now I don't use the Jaeger exporters, either. ;) |
another option would be to bump the version to |
Creating new 1.x release branches which mirror opentelemetry-java releases is interesting.. I'm also open to publishing patch releases for 1.34.x for security vulnerabilities until instrumentation's support for 1.x ends. I recall that semconv requires >= 6 months of support. Is there a specific support time period in mind for 1.x? |
oops, I meant |
I think we're open to security patching 1.x for longer than 6 months if there's demand |
I re-targeted this to the new |
Backport of #10455 and #10241