-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 metric mapping of counter metrics in the Openmetrics V2 version of the check #13573
Conversation
The |
1 similar comment
The |
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have an answer to extra_metrics
possibly not being honored (as mentioned in #12855 (comment))?
I tested it and I didn't have an issue adding an extra metric. For example:
with:
I get:
I can't reproduce the issue. |
What does this PR do?
Counters weren't getting collected because Openmetrics v2 expects counter metrics to be specified without the
_total
. This PR removes_total
from all the counter metrics in the openmetrics version of the check. This check never had a openmetrics v1 implementation. So I figured I can edit the main metric dictionary.Also fixes one metric that was being mapped to the wrong datadog name
envoy_cluster_upstream_cx_tx_bytes_buffered