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

tidb-in-kubernetes: add local pv example #1623

Merged
merged 19 commits into from Nov 22, 2019

Conversation

anotherrachel
Copy link
Contributor

@anotherrachel anotherrachel commented Oct 31, 2019

What is changed, added or deleted?

This PR adds an example about how to mount disks, modify and create local-volume-provisioner accordingly to use local disks for monitor, backup, binlog, pd and tikv, etc.

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

pingcap/docs-cn#1808
pingcap/docs-cn#2035

Which version does your change affect?

dev, v3.0, v3.1

@anotherrachel anotherrachel added area/tidb-in-k8s-v1.0 dev translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. labels Oct 31, 2019
@anotherrachel
Copy link
Contributor Author

@DanielZhangQD @yikeke PTAL, thanks!

@lilin90
Copy link
Member

lilin90 commented Nov 5, 2019

@anotherrachel Please address comments.

@anotherrachel
Copy link
Contributor Author

@DanielZhangQD All comments addressed. PTAL again, thanks!

@lilin90
Copy link
Member

lilin90 commented Nov 11, 2019

@yikeke @DanielZhangQD PTAL again~
@anotherrachel Please also apply changes to v3.1 when the content is ready. Thanks!

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

Same for v3.0~

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

Please apply the changes in pingcap/docs-cn#2035 to this PR~ @anotherrachel

@anotherrachel
Copy link
Contributor Author

Please apply the changes in pingcap/docs-cn#2035 to this PR~ @anotherrachel

Aligning

@anotherrachel
Copy link
Contributor Author

@yikeke @DanielZhangQD Changes in #2035 have been applied. PTAL again, thanks!

Copy link
Contributor

@DanielZhangQD DanielZhangQD left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90
Copy link
Member

lilin90 commented Nov 15, 2019

@yikeke PTAL again~

Copy link
Contributor

@DanielZhangQD DanielZhangQD left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90
Copy link
Member

lilin90 commented Nov 20, 2019

@anotherrachel Please also help apply changes to v3.1. @yikeke PTAL again.

@anotherrachel
Copy link
Contributor Author

@yikeke All comments addressed. If the content is ready, I'll update other versions soon. PTAL again, thanks~

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

LGTM

@yikeke yikeke merged commit 357dde2 into pingcap:master Nov 22, 2019
@anotherrachel anotherrachel deleted the add-local-pv-example-new branch November 25, 2019 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tidb-in-k8s-v1.0 translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants