-
Notifications
You must be signed in to change notification settings - Fork 778
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
Ensure OpenTelemetry continues to work with .NET 6 #2064
Comments
FYI @vishweshbankwar |
For reference: one of the behavior changes is noted here - dotnet/runtime#53375 |
https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry.Instrumentation.AspNetCore/OpenTelemetry.Instrumentation.AspNetCore.csproj#L3 Need to confirm if this continues to work in .NET6. |
Hi, is this the best Issue to watch to confirm compatibility with .NET 6? .NET 6 RC1 was released week, which is a stable production supported pre-release. |
Yes. As mentioned in the description, our dependencies are on runtime packages, which are not expected to contain any backward incompatible changes. This issue is to mostly keep track of testing, (and also to leverage some new capabilities in .NET 6). |
Opening an issue to track the work of making sure that this SDK continues to work well with .NET 6 as well. We have dependency on DiagnosticSource and Logging package from .NET runtime, and they are not expected to contain any breaking changes. However, it must be tested to make sure this is the case.
We need to try PRs (eg: #2057) with .NET 6 preview bits and see if there are CI failures. If there are behaviour change from .NET 6, it must be investigated and raised in .NET runtime repo.
Once we reach ~Sep-Oct timeframe when .NET 6 is nearing stable release, we can have the main branch release a preview of Otel , which refers the preview .NET 6 bits. (or we may piggyback on metrics branch which will always have .NET 6 dependency)
The text was updated successfully, but these errors were encountered: