-
Notifications
You must be signed in to change notification settings - Fork 28
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
prometheus exporter: use explicit histograms (#777)
The current prometheus exporter can't export histograms. This is because the current converter passes the prometheus exporter _exponential histograms_, which it doesn't know how to accept. This PR teaches the converter to produce _explicit histograms_ instead, which the prometheus exporter does know how to accept. Testing: * a simple integration test to makes sure the `/metrics` endpoint can produce is exposing histogram points * fairly involved unit tests of the converter from lightstep histogram to OTel explicit histogram R: @jmacd --------- Co-authored-by: Isaac Zinda <[email protected]> Co-authored-by: Joshua MacDonald <[email protected]>
- Loading branch information
1 parent
8c0a5bf
commit ec020ef
Showing
5 changed files
with
341 additions
and
15 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.