-
Notifications
You must be signed in to change notification settings - Fork 119
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
en: update doc for deploying TiDB cluster on ACK #62
Conversation
@@ -334,34 +421,6 @@ You can customize this script. For example, you can remove the `module "bastion" | |||
> | |||
> You can copy the `deploy/aliyun` directory. But you cannot copy a directory on which the `terraform apply` operation is currently performed. In this case, it is recommended to clone the repository again and then copy it. | |||
|
|||
## Destroy |
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.
Please keep this section here, no need to move to the front.
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.
This is to keep it consistent with the Chinese doc. Please confirm whether we should move it.
Co-Authored-By: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com>
Co-Authored-By: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com>
@DanielZhangQD PTAL again, thanks! |
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.
Rest LGTM
Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>
@yikeke @DanielZhangQD PTAL |
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
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
* en: update doc for deploying TiDB cluster on ACK * Apply suggestions from Daniel Co-Authored-By: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> * address comments from Daniel * Apply suggestions from code review Co-Authored-By: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> * address comments from Coco Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com> * move the destroy section Co-authored-by: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
cherry pick to release-1.1 in PR #105 |
* en: update doc for deploying TiDB cluster on ACK * Apply suggestions from Daniel Co-Authored-By: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> * address comments from Daniel * Apply suggestions from code review Co-Authored-By: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> * address comments from Coco Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com> * move the destroy section Co-authored-by: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com> Co-authored-by: Ran <huangran@pingcap.com> Co-authored-by: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
What is changed, added or deleted? (Required)
Update doc for deploying TiDB cluster on Alibaba cloud.
Which TiDB Operator version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?