-
Notifications
You must be signed in to change notification settings - Fork 720
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
metrics: add more metrics for region and store heartbeat #2881
metrics: add more metrics for region and store heartbeat #2881
Conversation
c707e56
to
9952e16
Compare
Hi, would you mind taking a look to see if this pr meets your expectations ? |
Signed-off-by: Zheming Li <nkdudu@126.com>
aca4b30
to
5f16212
Compare
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.
rest LGTM
plz sign dco |
@lhy1024 Should I also squash the commits ? |
it is not necessary to squash the commits, only to sign dco is enough. It will be squashed when merging pr |
fbc90e2
to
b568a5b
Compare
Signed-off-by: Zheming Li <nkdudu@126.com>
b568a5b
to
87b7530
Compare
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.
the rest LGTM, would you like to show the new panel with a screenshot?
Signed-off-by: Zheming Li <nkdudu@126.com>
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.
Thanks!
/merge |
/run-all-tests |
@duduainankai merge failed. |
/merge |
/run-all-tests |
@duduainankai merge failed. |
/run-integration-lightning-test |
2 similar comments
/run-integration-lightning-test |
/run-integration-lightning-test |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #2891 |
Signed-off-by: Zheming Li nkdudu@126.com
What problem does this PR solve?
Add more metrics about heartbeat. Solve #2800
What is changed and how it works?
Add heartbeat handle duration for region heartbeat and store heartbeat.
Add panel in pd grafana dashboard.
Fix RegionHeartBeat still count "ok" when error occurs.
Check List
Tests
Code changes
Release note