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

BR: Add backup-and-restore-faq.md #2686

Merged
merged 12 commits into from
Jun 5, 2020
Merged

BR: Add backup-and-restore-faq.md #2686

merged 12 commits into from
Jun 5, 2020

Conversation

toutdesuite
Copy link
Contributor

@toutdesuite toutdesuite commented Jun 4, 2020 •
edited
Loading

What is changed, added or deleted? (Required)

Create backup-and-restore-faq.md

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

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

If you select two or more versions from above, to trigger the bot to cherry-pick this PR to your desired release version branch(es), you must add corresponding labels such as needs-cherry-pick-4.0, needs-cherry-pick-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.

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

@toutdesuite toutdesuite added needs-cherry-pick-3.1 translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. labels Jun 4, 2020
@toutdesuite toutdesuite marked this pull request as draft June 4, 2020 15:05
@toutdesuite toutdesuite changed the title BR: Create backup-and-restore-faq.md BR: Add backup-and-restore-faq.md Jun 5, 2020
@toutdesuite toutdesuite marked this pull request as ready for review June 5, 2020 06:33
@toutdesuite toutdesuite requested review from yikeke and ran-huang June 5, 2020 06:34
@toutdesuite
Copy link
Contributor Author

@YuJuncen PTAL, thanks

YuJuncen
YuJuncen approved these changes Jun 5, 2020
View reviewed changes
Copy link
Contributor

@YuJuncen YuJuncen left a comment

Choose a reason for hiding this comment

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

LGTM

YuJuncen
YuJuncen reviewed Jun 5, 2020
View reviewed changes
br/backup-and-restore-faq.md Outdated

## What should I do to resolve the `Permission denied` error, even if I have tried to run BR using root in vain?

You need to confirm whether TiKV has access to the backup directory. To back up data, confirm whether TiKV has write privilege. To restore data, confirm whether it has read privilege.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe use permission would be better here, since the problem is "Permission denied"...?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in 123cb26~

@toutdesuite
Copy link
Contributor Author

@ran-huang PTAL, thanks~

ran-huang
ran-huang reviewed Jun 5, 2020
View reviewed changes
br/backup-and-restore-faq.md Outdated Show resolved Hide resolved
br/backup-and-restore-faq.md Outdated

This document lists the Frequently Asked Questions (FAQ) and the solutions about Backup & Restore (BR).

If the problems are not covered in this document and cannot be resolved, you can raise questions in the [AskTUG](http://asktug.com) community.
Copy link
Contributor

Choose a reason for hiding this comment

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

AskTUG does not have an English interface. I think it's not appropriate to direct English readers to AskTUG. Maybe it's better to put here a link to the BR repo so users can create issues.
@YuJuncen @toutdesuite What do you think?

br/backup-and-restore-faq.md Outdated Show resolved Hide resolved
br/backup-and-restore-faq.md Outdated Show resolved Hide resolved
br/backup-and-restore-faq.md Outdated Show resolved Hide resolved
toutdesuite and others added 2 commits June 5, 2020 19:01
@toutdesuite @ran-huang
Update br/backup-and-restore-faq.md
f9ea193 
Co-authored-by: Ran <huangran@pingcap.com>
@toutdesuite @ran-huang
Update br/backup-and-restore-faq.md
6966619 
Co-authored-by: Ran <huangran@pingcap.com>
@yikeke
Copy link
Contributor

yikeke commented Jun 5, 2020

Please ping me to add this file to TOC when it's ready to merge. @toutdesuite

toutdesuite and others added 2 commits June 5, 2020 20:25
@toutdesuite @ran-huang
Update br/backup-and-restore-faq.md
60a6921 
Co-authored-by: Ran <huangran@pingcap.com>
@toutdesuite @ran-huang
Update br/backup-and-restore-faq.md
8cb8f24 
Co-authored-by: Ran <huangran@pingcap.com>
ran-huang
ran-huang approved these changes Jun 5, 2020
View reviewed changes
Copy link
Contributor

@ran-huang ran-huang left a comment

Choose a reason for hiding this comment

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

LGTM

@toutdesuite
Copy link
Contributor Author

I' ll merge this PR when the link http://asktug.com is removed~

@yikeke
Copy link
Contributor

yikeke commented Jun 5, 2020

Please add the file to TOC and I'll review it. @toutdesuite

@toutdesuite
Copy link
Contributor Author

Please add the file to TOC and I'll review it. @toutdesuite

@yikeke done

@yikeke
Copy link
Contributor

yikeke commented Jun 5, 2020

TOC updated now. You can merge now. @toutdesuite

@toutdesuite toutdesuite added the status/can-merge Indicates a PR has been approved by a committer. label Jun 5, 2020
@ran-huang
Copy link
Contributor

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Jun 5, 2020

/run-all-tests

@sre-bot sre-bot merged commit b1ebd08 into pingcap:master Jun 5, 2020
sre-bot pushed a commit to sre-bot/docs that referenced this pull request Jun 5, 2020
@toutdesuite @sre-bot
cherry pick pingcap#2686 to release-3.1
159ce70 
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot sre-bot mentioned this pull request Jun 5, 2020
Merged
5 tasks
@sre-bot
Copy link
Contributor

sre-bot commented Jun 5, 2020

cherry pick to release-3.1 in PR #2714

sre-bot pushed a commit to sre-bot/docs that referenced this pull request Jun 5, 2020
@toutdesuite @sre-bot
cherry pick pingcap#2686 to release-4.0
a8b0543 
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot sre-bot mentioned this pull request Jun 5, 2020
Merged
5 tasks
@sre-bot
Copy link
Contributor

sre-bot commented Jun 5, 2020

cherry pick to release-4.0 in PR #2715

sre-bot added a commit that referenced this pull request Jun 5, 2020
@sre-bot
BR: Add backup-and-restore-faq.md (#2686) (#2715)
9c6d15b 
Signed-off-by: sre-bot <sre-bot@pingcap.com>
toutdesuite added a commit that referenced this pull request Jun 5, 2020
@sre-bot @toutdesuite
@ran-huang @yikeke
BR: Add backup-and-restore-faq.md (#2686) (#2714)
deb5cce 
* cherry pick #2686 to release-3.1

Signed-off-by: sre-bot <sre-bot@pingcap.com>

* resolve conflict

* Update TOC.md

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Ran <huangran@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YuJuncen YuJuncen YuJuncen approved these changes

@ran-huang ran-huang ran-huang approved these changes

@yikeke yikeke Awaiting requested review from yikeke

Assignees
No one assigned
Labels
status/can-merge Indicates a PR has been approved by a committer. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@toutdesuite @yikeke @ran-huang @sre-bot @YuJuncen