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

A sample/example as how to use Summaries #4088

Closed
SamYuan1990 opened this issue May 14, 2023 · 2 comments
Closed

A sample/example as how to use Summaries #4088

SamYuan1990 opened this issue May 14, 2023 · 2 comments
Labels
enhancement New feature or request pkg:exporter:prometheus Related to the Prometheus exporter package
Milestone

Comments

@SamYuan1990
Copy link

Problem Statement

A clear and concise description of what the problem is.
Ex. I'm always frustrated when I want to use Summaries ref https://opentelemetry.io/docs/specs/otel/compatibility/prometheus_and_openmetrics/#summaries but found https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/data-model.md#summary-legacy

Proposed Solution

A clear and concise description of what you want to happen.
I am confused as can I use Summaries? how can I use it? if not, any alteratives?

Alternatives

A clear and concise description of any alternative solutions or features you've considered.
n/A?

Prior Art

A clear and concise list of any similar and existing solutions from other projects that provide context to possible solutions.
n/A?

Additional Context

Add any other context or screenshots about the feature request here.
n/A?

@SamYuan1990 SamYuan1990 added the enhancement New feature or request label May 14, 2023
@SamYuan1990 SamYuan1990 changed the title A sample as how to use Summaries A sample/example as how to use Summaries May 14, 2023
@MrAlias MrAlias added the pkg:exporter:prometheus Related to the Prometheus exporter package label Sep 14, 2023
@dashpole
Copy link
Contributor

You cannot use summaries with the OpenTelemetry API, and I don't believe there is interest in adding API surface to include summaries. It would need to be added to the specification in order for us to support it in this repository.

We recommend using a Histogram instrument instead, e.g. with Float64Histogram():

Float64Histogram(name string, options ...Float64HistogramOption) (Float64Histogram, error)

If you really want a summary, it will be possible for bridges, such as the OpenCensus bridge or Prometheus bridge to produce summary data after #4622. This means will eventually be possible to use the prometheus client_golang to create a Summary, and add it to metrics exported by OpenTelemetry using the Prometheus bridge.

@dashpole dashpole moved this from Todo to In Progress in Go: Metric Prometheus Exporter (GA) Oct 20, 2023
@dashpole
Copy link
Contributor

Closing as not planned. I'm happy to answer further questions, but I don't think the OTel Go API is going to support summaries.

@XSAM XSAM added this to the untracked milestone Nov 7, 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 pkg:exporter:prometheus Related to the Prometheus exporter package
Development

No branches or pull requests

4 participants