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

enables prometheus exemplars support #1768

Merged
merged 1 commit into from
Feb 28, 2024
Merged

enables prometheus exemplars support #1768

merged 1 commit into from
Feb 28, 2024

Conversation

vroldanbet
Copy link
Contributor

enabling exemplars makes possible to correlate
traces with a metric. Clients scraping will be
able to do content negotiation and request
OpenMetrics format.

This change is not innocuous: prometheus defaults
to scraping with OpenMetrics format, and as a result the "le" labels of histograms will have a trailing zero. From the prometheus/client_golang docs

If true, the experimental OpenMetrics encoding is added to the
possible options during content negotiation. Note that Prometheus
2.5.0+ will negotiate OpenMetrics as first priority. OpenMetrics is
the only way to transmit exemplars. However, the move to OpenMetrics
is not completely transparent. Most notably, the values of "quantile"
labels of Summaries and "le" labels of Histograms are formatted with
a trailing ".0" if they would otherwise look like integer numbers
(which changes the identity of the resulting series on the Prometheus
server).

enabling exemplars makes possible to correlate
traces with a metric. Clients scraping will be
able to do content negotiation and request
OpenMetrics format.

This change is not innocuous: prometheus defaults
to scraping with OpenMetrics format, and as a result
the "le" labels of histograms will have a trailing zero.
From the prometheus/client_golang docs

// If true, the experimental OpenMetrics encoding is added to the
// possible options during content negotiation. Note that Prometheus
// 2.5.0+ will negotiate OpenMetrics as first priority. OpenMetrics is
// the only way to transmit exemplars. However, the move to OpenMetrics
// is not completely transparent. Most notably, the values of "quantile"
// labels of Summaries and "le" labels of Histograms are formatted with
// a trailing ".0" if they would otherwise look like integer numbers
// (which changes the identity of the resulting series on the Prometheus
// server).
@vroldanbet vroldanbet requested a review from a team as a code owner February 27, 2024 09:29
@vroldanbet vroldanbet changed the title enables exemplars prometheus support enables prometheus exemplars support Feb 27, 2024
@github-actions github-actions bot added area/CLI Affects the command line area/dependencies Affects dependencies labels Feb 27, 2024
@vroldanbet vroldanbet added this pull request to the merge queue Feb 28, 2024
Merged via the queue into main with commit 5a32447 Feb 28, 2024
20 checks passed
@vroldanbet vroldanbet deleted the enable-exemplars branch February 28, 2024 18:01
@github-actions github-actions bot locked and limited conversation to collaborators Feb 28, 2024
@jzelinskie jzelinskie added the area/observability Affects telemetry data label Feb 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/CLI Affects the command line area/dependencies Affects dependencies area/observability Affects telemetry data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants