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

fix histogram metrics in OTLP #2564

Merged
merged 5 commits into from
Feb 15, 2023
Merged

fix histogram metrics in OTLP #2564

merged 5 commits into from
Feb 15, 2023

Conversation

Geal
Copy link
Contributor

@Geal Geal commented Feb 7, 2023

Fix #2493

with the inexpensive selector, histogram are only reported as gauges, and so they will be incorrectly interpreted when reaching Datadog

with the inexpensive selector, histogram are only reported as gauges,
and so they will be incorrectly interpreted when reaching Datadog
@github-actions

This comment has been minimized.

@Geal
Copy link
Contributor Author

Geal commented Feb 8, 2023

I'm not fixed on the list of buckets we actually push, we can change it (I think the higher buckets, at more than 1mn are less useful)

@Geal Geal requested review from BrynCooke and SimonSapin February 8, 2023 14:05
@@ -41,8 +42,33 @@ impl MetricsConfigurator for super::super::otlp::Config {
Some(exporter) => {
let exporter = opentelemetry_otlp::new_pipeline()
.metrics(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might make sense to add a comment for this, as to why inexpensive() selector is not used.

@Geal Geal merged commit 47bf502 into dev Feb 15, 2023
@Geal Geal deleted the geal/fix-datadog-histograms branch February 15, 2023 09:19
@abernix abernix mentioned this pull request Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

apollo_router_processing_time is ever increasing
3 participants