Skip to content

Prometheus exponential histogram validate scale range #6779

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.

Sign up for GitHub

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

Jump to bottom
Open
dashpole opened this issue May 15, 2025 · 1 comment · May be fixed by #6822
Open

Prometheus exponential histogram validate scale range #6779

dashpole opened this issue May 15, 2025 · 1 comment · May be fixed by #6822
Assignees
@highlyavailable
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Milestone
v1.37.0

Comments

@dashpole
Copy link
Contributor

I'm doing a late review of #6421, and am comparing it to the implementation here: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/9cb3ada6449ae195aa904f82f1ac972ff401974b/pkg/translator/prometheusremotewrite/histograms.go#L50

We need to ensure the scale is valid for Prometheus (-4 to 8), and scale down if it is greater than 8, or reject.

cc @GiedriusS
@MrAlias MrAlias added help wanted Extra attention is needed good first issue Good for newcomers labels May 22, 2025
@MrAlias MrAlias added this to the v1.37.0 milestone May 22, 2025
@highlyavailable
Copy link
Contributor

@MrAlias could I pick this up?

highlyavailable added a commit to highlyavailable/opentelemetry-go that referenced this issue May 24, 2025
@highlyavailable
prometheus: validate exponential histogram scale range (open-telemetr…
e90f741 
…y#6779)
@highlyavailable highlyavailable mentioned this issue May 24, 2025
Closed
highlyavailable added a commit to highlyavailable/opentelemetry-go that referenced this issue May 24, 2025
@highlyavailable
prometheus: validate exponential histogram scale range (open-telemetr…
6f10f59 
…y#6779)
@highlyavailable highlyavailable linked a pull request May 24, 2025 that will close this issue
Open
highlyavailable added a commit to highlyavailable/opentelemetry-go that referenced this issue May 24, 2025
@highlyavailable
prometheus: validate exponential histogram scale range (open-telemetr…
4c1e41b 
…y#6779)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@highlyavailable highlyavailable

Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
v1.37.0
3 participants
@dashpole @MrAlias @highlyavailable