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.

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

Opentelemetry OTLP support #652

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Opentelemetry OTLP support #652

wants to merge 1 commit into from

Conversation

joepio
Copy link
Member

@joepio joepio commented Aug 21, 2023

We already have support for Jaeger using opentelemetry_rust, but not yet for other opentelemetry implementations like OpenObserve. Would be nice to switch to OTLP. We could probably ditch opentelemetry_jaeger and only use opentelemetry_otlp.

However, I had some issues while implementing. I need an HTTP client like hyper or reqwest. Not a lot of docs for this. I think I'm close though.

PR Checklist:

  • Link to related issues: #number
  • Add changelog entry linking to issue, describe API changes
  • Add tests (optional)
  • (If new feature) add to description / readme

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant