open-telemetry / opentelemetry-dotnet-contrib

This repository contains set of components extending functionality of the OpenTelemetry .NET SDK. Instrumentation libraries, exporters, and other components can find their home here.
https://opentelemetry.io
Apache License 2.0
449 stars 273 forks source link

[Instrumentation.Runtime] Future plans #2071

Open joegoldman2 opened 1 week ago

joegoldman2 commented 1 week ago

With the introduction of built-in runtime metrics in .NET 9 (https://github.com/dotnet/runtime/pull/104680), I wanted to understand the future plans for OpenTelemetry.Instrumentation.Runtime. Specifically:

Kielek commented 5 days ago

@twenzel, @xiang17 FYI

Kielek commented 5 days ago

I would say, that we should keep this package.

For .NET Framework and .NET8 (I assume that .NET6 will be removed) we should keep current behavior. For .NET9+ we should register System.Runtime in meter.

We have similar behavior for AspNetCore already.

twenzel commented 4 days ago

@Kielek sounds good to me.

xiang17 commented 3 days ago

Agree to keep current behavior for existing use cases.