Skip to content
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: update monitoring template in grafana #2442

Merged
merged 3 commits into from
May 22, 2020

Conversation

nolouch
Copy link
Contributor

@nolouch nolouch commented May 22, 2020

Signed-off-by: nolouch nolouch@gmail.com

What problem does this PR solve?

  • Add TSO handle duration in server-side
  • Add Heartbeat QPS for update_kv and update_cache
  • Add direction of balance hot region
  • Add the hot cache entry numbers
  • Fix balance Region movement
  • Add Rule checker

What is changed and how it works?

Update promQL

Check List

Tests

  • Mannual test
    image

Release note

  • No

Signed-off-by: nolouch <nolouch@gmail.com>
@nolouch nolouch added component/metrics Metrics. needs-cherry-pick-release-4.0 The PR needs to cherry pick to release-4.0 branch. labels May 22, 2020
Copy link
Contributor

@lhy1024 lhy1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would you like to show the new one via screenshot?

metrics/grafana/pd.json Outdated Show resolved Hide resolved
metrics/grafana/pd.json Outdated Show resolved Hide resolved
metrics/grafana/pd.json Outdated Show resolved Hide resolved
metrics/grafana/pd.json Outdated Show resolved Hide resolved
"timeFrom": null,
"timeRegions": [],
"timeShift": null,
"title": "Direction of hotspot move leader",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The direction is only in hot write row.

Copy link
Contributor Author

@nolouch nolouch May 22, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes. I think it's about the total hot region rather than hot read or hot write. But I cannot find a suitable place.

Signed-off-by: nolouch <nolouch@gmail.com>
@lhy1024
Copy link
Contributor

lhy1024 commented May 22, 2020

/approve

Copy link
Contributor

@sre-bot sre-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HunDunDM
Copy link
Member

/approve

Copy link
Contributor

@sre-bot sre-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nolouch
Copy link
Contributor Author

nolouch commented May 22, 2020

/merge

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label May 22, 2020
@sre-bot
Copy link
Contributor

sre-bot commented May 22, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented May 22, 2020

cherry pick to release-4.0 in PR #2448

jebter added a commit that referenced this pull request May 25, 2020
* metrics: update json template in grafana

Signed-off-by: nolouch <nolouch@gmail.com>

* address comments

Signed-off-by: nolouch <nolouch@gmail.com>

Co-authored-by: nolouch <nolouch@gmail.com>
Co-authored-by: jebter <jebter@126.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/metrics Metrics. needs-cherry-pick-release-4.0 The PR needs to cherry pick to release-4.0 branch. status/can-merge Indicates a PR has been approved by a committer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants