-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
config: correct the explanation of status and metrics. #13024
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13024 +/- ##
===========================================
Coverage 80.0635% 80.0635%
===========================================
Files 468 468
Lines 110160 110160
===========================================
Hits 88198 88198
Misses 15187 15187
Partials 6775 6775 |
LGTM |
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.
LGTM
cherry pick to release-3.1 in PR #13107 |
cherry pick to release-3.0 in PR #13108 |
cherry pick to release-2.1 failed |
It seems that, not for sure, we failed to cherry-pick this commit to release-2.1. Please comment '/run-cherry-picker' to try to trigger the cherry-picker if we did fail to cherry-pick this commit before. @ericsyh PTAL. |
/run-cherry-picker |
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-2.1 in PR #16179 |
What problem does this PR solve?
The explanation of
metrics-addr
andstatus-port
is not so accurate.status-port
copiedmetrics-addr
explanation which is a mistake, need to remove it.metrics-addr
leaves empty will disable the push to Pushgateway, rather than Prometheus push.What is changed and how it works?
status-port
status-host
andstatus-port
metrics-addr
Check List
Tests
Related changes