feat: add metric name translation strategy support for the Prometheus exporter#5118
feat: add metric name translation strategy support for the Prometheus exporter#5118herin049 wants to merge 8 commits into
Conversation
MikeGoldsmith
left a comment
There was a problem hiding this comment.
Looks great, thanks @herin049 🎉
125513f to
79162b3
Compare
79162b3 to
ca66756
Compare
|
This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment. |
8ecb28d to
4d4c951
Compare
|
Small update on this PR, after going through the PR again and adding additional tests, I came to realize that the Python Prometheus client package ( Likewise, there it also has the behavior of always appending the We will likely have to put this particular feature on hold until the required changes are made to the |
Description
Add support for specifying a name translation strategy for the Prometheus exporter as documented in: opentelemetry-specification
Fixes #5111
Depends on: #5123
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Does This PR Require a Contrib Repo Change?
Checklist: