-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
how-to/monitor: update manual deploy monitor #1965
Conversation
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 changes in this PR is different from that comment in Is there a file tivk_pull.json? tidb-ansible#999. Please check whether we need to add all JSON files mentioned in that comment. For example,
tikv_trouble_shooting.json
is missing. - The
best-practices
label only applies to best practice articles. This is not one, so you don't need to add that label.
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
You are right, |
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
@liubo0127 I think you might need a technical review? |
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
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
What is changed, added or deleted?
Update the dashboard file for manual deploy monitor. In master or 3.0, some file does not exist.
What is the related PR or file link(s)?
pingcap/tidb-ansible#999
#1962
Which version does your change affect?
Affects TiDB master and 3.0.