-
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
Update the configuration to include the metric_prefix
option
#17065
Conversation
a083abf
to
e088a8d
Compare
The |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more. |
Test Results 105 files 105 suites 1h 21m 58s ⏱️ Results for commit 165bcbf. ♻️ This comment has been updated with latest results. |
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.
looks good for docs
* Update the configuration to include the `metric_prefix` option * Update the configuration to include the `metric_prefix` option
What does this PR do?
Update the configuration to include the
metric_prefix
optionMotivation
Added in the base check in #16958
Additional Notes
Noticed we forgot it working on a support case
Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.