Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove testing of jaeger exporter #3508

Closed
ocelotl opened this issue Nov 4, 2023 · 2 comments · Fixed by #3554
Closed

Remove testing of jaeger exporter #3508

ocelotl opened this issue Nov 4, 2023 · 2 comments · Fixed by #3554
Assignees
@ocelotl

Comments

@ocelotl
Copy link
Contributor

ocelotl commented Nov 4, 2023 

          This will cause a conflict with packages/applications trying to install newer protobuf 4. Similar to https://github.com/open-telemetry/opentelemetry-python/issues/3074

Originally posted by @aabmass in #3502 (comment)
@ocelotl
Copy link
Contributor Author

ocelotl commented Nov 4, 2023

As per this comment, I suggest we remove testing for this package.

@ocelotl ocelotl self-assigned this Nov 4, 2023
@aabmass
Copy link
Member

aabmass commented Nov 15, 2023

Should we just remove the package altogether and stop making new releases of it? It feels weird to have code in our repo that we aren't testing but still releasing it.

Also does this relate directly to my original comment in the OP?

ocelotl added a commit to ocelotl/opentelemetry-python that referenced this issue Nov 22, 2023
@ocelotl
Remove Jaeger exporter
ed98861 
Fixes open-telemetry#3514
Fixes open-telemetry#3508
@ocelotl ocelotl mentioned this issue Nov 22, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ocelotl ocelotl

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Jaeger exporter ocelotl/opentelemetry-python
2 participants
@aabmass @ocelotl