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

Promote MetricPoint reclaim feature for delta aggregation from experimental to stable #5443

Closed
CodeBlanch opened this issue Mar 13, 2024 · 5 comments · Fixed by #5956
Closed
Labels
metrics Metrics signal related pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package
Milestone

Comments

@CodeBlanch
Copy link
Member

For the experimental feature see: #2360

This issue is for tracking making this a stable feature.

@CodeBlanch CodeBlanch added pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package metrics Metrics signal related labels Mar 13, 2024
@CodeBlanch CodeBlanch added this to the Future milestone Mar 13, 2024
@utpilla utpilla modified the milestones: Future, 1.9.0 Mar 13, 2024
@vchirikov
Copy link

vchirikov commented May 5, 2024

we have a 17.4 MB endpoint "/metrics" (otel -> prometheus exporter) because of the old metrics, waiting for this feature. Is there any news about this?

p.s. I will try to use OTEL_DOTNET_EXPERIMENTAL_METRICS_RECLAIM_UNUSED_METRIC_POINTS and send the results looks like #5546 should be released first

@cijothomas
Copy link
Member

we have a 17.4 MB endpoint "/metrics" (prometheus exporter) because of the old metrics, waiting for this feature. Is there any news about this?

p.s. I will try to use OTEL_DOTNET_EXPERIMENTAL_METRICS_RECLAIM_UNUSED_METRIC_POINTS and send the results looks like #5546 should be released first

The feature tracked by this is only for DELTA... If you are using Prometheus, then this won't help you, as prometheus only accepts Cumulative.

@vishweshbankwar vishweshbankwar modified the milestones: 1.9.0, Future May 21, 2024
@yoziv
Copy link

yoziv commented Jul 26, 2024

When is this expected ?

@cijothomas
Copy link
Member

When is this expected ?

There is no ETA for this. If you are using the experimental feature, we'd love to get any feedback you have on the feature. We don't have enough feedback to confidently promote this feature to stable. It is also possible that the current approach could be revisited to make it less complex.

@CodeBlanch CodeBlanch modified the milestones: Future, 1.10.0 Nov 6, 2024
@CodeBlanch
Copy link
Member Author

We discussed this on the SIG meeting today. We're doing a final review and some testing of the feature but the plan is to remove the envvar and turn it on by default for 1.10.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metrics Metrics signal related pkg:OpenTelemetry Issues related to OpenTelemetry NuGet package
Projects
None yet
6 participants