open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
2.97k stars 2.3k forks source link

improve summary metrics converting. #33850

Open h0cheung opened 3 months ago

h0cheung commented 3 months ago

Component(s)

processor/transform

Is your feature request related to a problem? Please describe.

There are functions like convert_summary_count_val_to_sum and convert_summary_sum_val_to_sum, which can convert values in summary to sum and count. However, the have some limitions:

Describe the solution you'd like

  1. Update function specification: convert_summary_count_val_to_sum(aggregation_temporality, is_monotonic, Optional[suffix]). The new optional parameters allow setting the suffix to add. By default, suffix is ".sum".
  2. New function: converter_summary_quantile_val_to_gauge(Optional[suffix]). The suffix defaults to ".quantile_%s", where %s will be replaced with the quantile value and the decimal point will be removed. For example, ".quantile_99", ".quantile_999" will be added.

Describe alternatives you've considered

Use a single function or component to convert summary to two sums and a gauge.

Additional context

No response

github-actions[bot] commented 3 months ago

Pinging code owners:

github-actions[bot] commented 3 weeks ago

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 @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.