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

executor: add diagnosis rule to check some metrics exceed thresholds #14801

Merged
merged 13 commits into from
Feb 18, 2020

Conversation

crazycs520
Copy link
Contributor

Signed-off-by: crazycs crazycs520@gmail.com

What problem does this PR solve?

This PR adds a new diagnosis rule -- threshold-check, which is used to check some metrics threshold value, like tikv thread cpu usage. We will detect the following metrics tables in the current implementation:

  • tikv_thread_cpu

Consider the PR length, this PR only add some part checks of threshold-check rule, I will add more check in the next PR.

What is changed and how it works?

Check List

Tests

  • Unit test
  • Manual test

Release note

  • Add diagnosis rule threshold-check which is used to check some metrics threshold value.

Signed-off-by: crazycs <crazycs520@gmail.com>
@crazycs520 crazycs520 added the sig/execution SIG execution label Feb 14, 2020
executor/diagnostics.go Outdated Show resolved Hide resolved
executor/diagnostics.go Outdated Show resolved Hide resolved
executor/diagnostics.go Outdated Show resolved Hide resolved
executor/diagnostics_test.go Show resolved Hide resolved
executor/diagnostics.go Outdated Show resolved Hide resolved
executor/diagnostics.go Outdated Show resolved Hide resolved
executor/diagnostics.go Outdated Show resolved Hide resolved
executor/diagnostics_test.go Outdated Show resolved Hide resolved
crazycs520 and others added 4 commits February 17, 2020 19:56
Co-Authored-By: Maxwell <passagesfall@gmail.com>
Signed-off-by: crazycs <crazycs520@gmail.com>
Signed-off-by: crazycs <crazycs520@gmail.com>
@crazycs520
Copy link
Contributor Author

/rebuild

Copy link
Contributor

@Deardrops Deardrops left a comment

Choose a reason for hiding this comment

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

LGTM

@Deardrops Deardrops added the status/LGT1 Indicates that a PR has LGTM 1. label Feb 18, 2020
executor/diagnostics_test.go Outdated Show resolved Hide resolved
Signed-off-by: crazycs <crazycs520@gmail.com>
@crazycs520
Copy link
Contributor Author

/rebuild

@crazycs520
Copy link
Contributor Author

/rebuild

Copy link
Contributor

@lonng lonng left a comment

Choose a reason for hiding this comment

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

LGTM

@lonng
Copy link
Contributor

lonng commented Feb 18, 2020

/merge

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

sre-bot commented Feb 18, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Feb 18, 2020

@crazycs520 merge failed.

@lonng
Copy link
Contributor

lonng commented Feb 18, 2020

/rebuild

2 similar comments
@crazycs520
Copy link
Contributor Author

/rebuild

@crazycs520
Copy link
Contributor Author

/rebuild

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/execution SIG execution status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants