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.

Sign up for GitHub

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

Jump to bottom

Update dashboard FAQ #5522

Closed
wants to merge 7 commits into from
Closed

Update dashboard FAQ #5522

wants to merge 7 commits into from

Conversation

shhdgit
Copy link
Member

@shhdgit shhdgit commented Apr 27, 2021

First-time contributors' checklist

What is changed, added or deleted? (Required)

Added a new FAQ for TiDB Dashboard

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v5.0 (TiDB 5.0 versions)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@ti-chi-bot ti-chi-bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 27, 2021
@TomShawn
Copy link
Contributor

TomShawn commented Apr 27, 2021 •
edited
Loading

/label needs-cherry-pick-5.0
/label needs-cherry-pick-master
/translation from-docs-cn
/cc @breeswish
This PR is translated from pingcap/docs-cn#6178

@ti-chi-bot ti-chi-bot added translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. needs-cherry-pick-5.0 needs-cherry-pick-master Should cherry pick this PR to master branch. and removed missing-translation-status This PR does not have translation status info. labels Apr 27, 2021
@TomShawn
Copy link
Contributor

/hold

@ti-chi-bot ti-chi-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 27, 2021
@TomShawn
Copy link
Contributor

TomShawn commented May 6, 2021 •
edited
Loading

This PR is translated from pingcap/docs-cn#6178.

@shhdgit shhdgit mentioned this pull request May 26, 2021
Merged
@ti-chi-bot ti-chi-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jun 22, 2021
@shhdgit
Copy link
Member Author

shhdgit commented Jun 22, 2021

Updated, PTAL.

@TomShawn
Copy link
Contributor

@shhdgit Please help confirm whether the PR change applies to v5.1. Thanks~

@shhdgit
Copy link
Member Author

shhdgit commented Jun 22, 2021

Yes, v5.1 can also be applied.

@TomShawn
Copy link
Contributor

@breeswish PTAL again, thanks~

breezewish
breezewish approved these changes Jun 28, 2021
View reviewed changes
dashboard/dashboard-faq.md Outdated Show resolved Hide resolved
dashboard/dashboard-faq.md Outdated Show resolved Hide resolved
@ti-chi-bot
Copy link
Member

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • breeswish

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot added the status/LGT1 Indicates that a PR has LGTM 1. label Jun 28, 2021
shhdgit and others added 2 commits June 29, 2021 10:25
@shhdgit @breezewish
Update dashboard/dashboard-faq.md
410fc86 
Co-authored-by: Wenxuan <hi@breeswish.org>
@shhdgit
Update dashboard-faq.md
4265ede
@breezewish
Copy link
Member

@TomShawn PTAL, thanks!

TomShawn
TomShawn reviewed Jun 29, 2021
View reviewed changes
dashboard/dashboard-faq.md

### An `unknown field` error is shown in **Slow Queries** page

If the `unknown field` error appears on the Slow Queries page after the cluster upgrade, that is a compatibility issue caused by the difference between TiDB Dashboard server fields (which may be updated) and user preferences fields (which are in the browser cache). This issue has been fixed in v4.0.14/v5.0.3 and later, older versions can be resolved by clearing the browser Local Storage.
Copy link
Contributor

Choose a reason for hiding this comment

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

@shhdgit @breeswish This PR is based on release-4.0 and will be first merged into the v4.0 documentation. If it applies to v4.0.14, I am afraid the merge will be no earlier than the v4.0.14 release.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, good catch!! @shhdgit Could you propose a PR based on master instead?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure.

@shhdgit shhdgit changed the base branch from release-4.0 to master June 30, 2021 02:33
@shhdgit shhdgit changed the base branch from master to release-4.0 June 30, 2021 02:33
@shhdgit shhdgit mentioned this pull request Jun 30, 2021
Merged
12 tasks
@TomShawn
Copy link
Contributor

@shhdgit Since this PR content is migrated to #5895, shall we close this one?

@shhdgit
Copy link
Member Author

shhdgit commented Jun 30, 2021

Yes.

@shhdgit shhdgit closed this Jun 30, 2021
@TomShawn TomShawn removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-cherry-pick-master Should cherry pick this PR to master branch. needs-cherry-pick-release-5.0 size/S Denotes a PR that changes 10-29 lines, ignoring generated files. status/LGT1 Indicates that a PR has LGTM 1. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. labels Jun 30, 2021
@ti-chi-bot ti-chi-bot added status/LGT1 Indicates that a PR has LGTM 1. missing-translation-status This PR does not have translation status info. labels Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomShawn TomShawn TomShawn left review comments

@breezewish breezewish breezewish approved these changes

Assignees
No one assigned
Labels
missing-translation-status This PR does not have translation status info. status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@shhdgit @TomShawn @ti-chi-bot @breezewish