-
Notifications
You must be signed in to change notification settings - Fork 186
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
Feature - OpenTelemetry Exporter #130
Comments
@atrauzzi - are you planning on working on this effort? |
I'm not sure I'd be the best person for it as I'm not intimately familiar with the otel format itself. |
I think it may be worth it to wait until the spec is hashed out to implement this. Thoughts? |
Definitely logic to that. Also gives this library a chance to settle on its API. |
The OTEP maintainers are discussing this now: open-telemetry/opentelemetry-specification#699 |
The OTLP exporter is now part of the spec: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/protocol/exporter.md |
OTLP Trace Data format: https://github.com/open-telemetry/oteps/blob/master/text/trace/0059-otlp-trace-data-format.md |
The HTTP version of this has been completed in #210 @riticksingh is now working on the gRPC implementation. |
It probably goes without saying that every OpenTelemetry library should should ship an OpenTelemetry exporter. 😄
The benefit to the PHP library in this would be that at some point, pure OpenTelemetry configurations are going to be possible, and desirable. Having an "ecosystem-native" exporter ensures that the PHP library doesn't contribute any delay towards widespread adoption.
It also helps simplify setup for newcomers by not forcing them into ecosystems and terminology outside of OpenTelemetry.
The text was updated successfully, but these errors were encountered: