-
Notifications
You must be signed in to change notification settings - Fork 764
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
Http instrumentation libraries stable release plan #4484
Comments
FYI: this doc is useful for tracking changes to the conventions: |
FYI. Trask advised that we should link to "main" for now. I will create a new PR to correct everything that merged already. |
Are there any plans to publish the schema url? Is that something that would happen here or in the runtime? Consuming this data from many different language SDKs across different versions of those SDKs becomes an issue when querying data in various backends. I would love to start using the schema processor to help alleviate this but the schema url needs to be available. |
Not possible until this is done : #2417 |
open-telemetry/opentelemetry-specification#3443 lays out the detailed plan for stabilizing the http instrumentation libraries.
PRs that include major changes
open-telemetry/opentelemetry-specification#3402
open-telemetry/opentelemetry-specification#3426
open-telemetry/opentelemetry-specification#3355
For .NET, we will proceed as follows
OTEL_SEMCONV_STABILITY_OPT_IN
environment variable and set the attributes as per the values specified in spec.Note: The change impacts all the above libraries due to updates in networking attributes open-telemetry/opentelemetry-specification#3402
Release the RC version with the above mentioned changes.
Drop
OTEL_SEMCONV_STABILITY_OPT_IN
environment variable fromOpenTelemetry.Instrumentation.AspNetCore
andOpenTelemetry.Instrumentation.Http
in the final RC version.Note: Other libraries should continue to look up the environment variable.
OpenTelemetry.Instrumentation.AspNetCore
andOpenTelemetry.Instrumentation.Http
.The text was updated successfully, but these errors were encountered: