-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Compatibility metric type summary #30156
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
@maokitty A summary does not have an aggregation temporality field https://github.com/open-telemetry/opentelemetry-proto/blob/ea449ae0e9b282f96ec12a09e796dbb3d390ed4f/opentelemetry/proto/metrics/v1/metrics.proto#L245-L247 as opposed to all other metric types handled by the cumulative to delta processor. I think a solution here would be for something like the transform processor to allow you to split a summary into its constituent parts (count, sum, quantiles as separate metrics) and then apply cumulative-to-delta to them. |
Pinging code owners for processor/transform: @TylerHelmuth @kentquirk @bogdandrutu @evan-bradley. See Adding Labels via Comments if you do not have permissions to add labels yourself. |
There already are some summary conversion functions:
Are these helpful? |
I think they should be, maybe a quantiles one would be helpful as well but IMO we should wait until someone explicitly asks for it |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been closed as inactive because it has been stale for 120 days with no activity. |
Component(s)
processor/cumulativetodelta
Is your feature request related to a problem? Please describe.
As there is a summary in prometheus and opentelemetry-summary-legacy, convert it to delta for compatibility.
Describe the solution you'd like
Convert sum and count to delta and leave quantiles unchanged as quantile are time window based values.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: