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

Support .NET 9 Synchronous Gauge #4805

Open
tommaybe opened this issue Aug 28, 2023 · 2 comments
Open

Support .NET 9 Synchronous Gauge #4805

tommaybe opened this issue Aug 28, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request metrics need-runtime-change Issues which likely require changes from dotnet runtime - typically DiagnosticSource changes
Milestone

Comments

@tommaybe
Copy link

tommaybe commented Aug 28, 2023

Feature Request

It would be great for synchronous gauge support to be added, now that it has been added to Experimental spec.

Is your feature request related to a problem?

Use cases are described here: open-telemetry/opentelemetry-specification#2318.

Describe the solution you'd like

I would expect to have a method in the class Meter called CreateObservableGauge() along with the class Gauge. They should behave like the ObservableGauge instrument, but being synchronous instead of asynchronous.

This is described as an experimental feature in the OpenTelemetry specification: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#gauge

Describe alternatives you've considered

Both UpDownCounter and ObservableGauge do not work for my usecases.

Additional context

@tommaybe tommaybe added the enhancement New feature or request label Aug 28, 2023
@cijothomas cijothomas added metrics need-runtime-change Issues which likely require changes from dotnet runtime - typically DiagnosticSource changes labels Aug 28, 2023
@cijothomas
Copy link
Member

Added the runtime-change needed label, as the instruments must be added in .NET Runtime and then OTel .NET (this repo) need to add support. This will come in .NET 9 timeframe only as .NET 8 train already left. Will open tracking issue in runtime repo, and link to this.

@cijothomas
Copy link
Member

.NET Runtime proposal is here : dotnet/runtime#92625 (comment)

@CodeBlanch CodeBlanch changed the title Synchronous Gauge support Support .NET 9 Synchronous Gauge Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request metrics need-runtime-change Issues which likely require changes from dotnet runtime - typically DiagnosticSource changes
Projects
None yet
Development

No branches or pull requests

3 participants