Skip to content
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

Update OTEL dotnet mappings to include process information #409

Open
IGx89 opened this issue Nov 27, 2024 · 1 comment
Open

Update OTEL dotnet mappings to include process information #409

IGx89 opened this issue Nov 27, 2024 · 1 comment

Comments

@IGx89
Copy link

IGx89 commented Nov 27, 2024

OpenTelemetry.Instrumentation.Process adds some new process metrics following Semantic Convention v1.25.0:

  • process.cpu.time
  • process.memory.virtual
  • process.thread.count

There currently are no mappings for those metrics to their corresponding Datadog metrics:

  • runtime.dotnet.cpu.system
  • runtime.dotnet.cpu.user
  • runtime.dotnet.cpu.percent
  • runtime.dotnet.mem.committed
  • runtime.dotnet.threads.count

Is it possible to add that?

@IGx89
Copy link
Author

IGx89 commented Nov 29, 2024

Various OTEL process metrics are being renamed in .NET 9 it looks: open-telemetry/opentelemetry-dotnet-contrib#2339. See https://learn.microsoft.com/en-us/dotnet/core/diagnostics/built-in-metrics-runtime for the official docs on them. Given that, I'd recommend mapping these metric names instead:

  • dotnet.process.cpu.time
  • dotnet.process.memory.working_set
  • dotnet.thread_pool.thread.count

Other already-mapped metrics are renamed in .NET 9 too, requiring further updates to the mappings too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant