-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Rework .NET Docs - proposed mini-IA #1001
Comments
A few questions:
|
I think so, yes. At least certainly a terminology overview. The API reference itself is actually just here: https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics?view=net-6.0 ...there's nothing in there about OTel (in fact, the core
For .NET Framework, the main differences are in setup (where to "hook" in otel), and automatic instrumentation for ASP.NET (rather than ASP.NET Core, which doesn't run on .NET Framework). Everything else is the same OTel APIs, and exporters are even the same too (as far as I can tell). So I think it could just be a separate doc. |
I think I will close this one out. After sleeping on it a few times, I don't think there needs to be a dedicated terminology&APIs section. |
.NET is a little special. So I propose a mini-IA for the .NET docs:
Under instrumentation/.NET:
Most .NET developers doing modern services work use modern .NET (i.e., not .NET Framework, which is a legacy Windows-based technology).
So, I propose that everything except for a page entitled ".NET Framework" (or something like it) uses modern .NET APIs - ASP.NET Core, Microsoft.Extensions.Hosting, etc. And in that one doc, the things you need to do to configure things for .NET Framework-based apps is covered instead, rather than putting it as an option for every single concept in the docs.
Additionally, since .NET recommends using the
System.Diagnostics
API over the OpenTelemetry API, there should be a dedicated doc that explains the difference and maps concepts from one to the other in a table.Thoughts?
The text was updated successfully, but these errors were encountered: