improve serverless dms doc #15793
improve serverless dms doc #15793
Conversation
ti-chi-bot
bot
added
missing-translation-status
zhangyangyu
force-pushed
the
dms-serverless
branch
from
1acac4d to
bc52c98
Compare
zhangyangyu
force-pushed
the
dms-serverless
branch
from
bc52c98 to
4145e03
Compare
ti-chi-bot
bot
added
size/L
This reverts commit 4145e03.
zhangyangyu
force-pushed
the
dms-serverless
branch
from
759746c to
5232b93
Compare
ti-chi-bot
bot
removed
the
missing-translation-status
Co-authored-by: Grace Cai <[email protected]>
Co-authored-by: Grace Cai <[email protected]>
zhangyangyu
force-pushed
the
dms-serverless
branch
from
8a327eb to
30bc235
Compare
|
@shiyuhang0: adding LGTM is restricted to approvers and reviewers in OWNERS files. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
|
||
| ## Network configuration | ||
|
|
||
| Before creating DMS resources, you need to configure network properly to ensure DMS can communicate with TiDB Cloud clusters. If you are unfamiliar with AWS, contact AWS Support. We give several possible configurations here. |
There was a problem hiding this comment.
"If you are unfamiliar with AWS, contact AWS Support." 这里这么写不知道是否恰当，或者 If you are unfamiliar with AWS, see xxx documentation 之类的可以吗
There was a problem hiding this comment.
网络需要根据用户各自的情况去配置，没有一个好的文档可以看吧。
Co-authored-by: Grace Cai <[email protected]>
|
|
||
| ### An AWS account with enough access | ||
|
|
||
| You are expected to have an AWS account with enough access to manage DMS-related resources. If not, refer to the following AWS documents: |
There was a problem hiding this comment.
"enough access" is a bit ambiguous. Could you list the specific permissions or roles needed for this AWS account or link to AWS documentation?
There was a problem hiding this comment.
It's the second link below.
Co-authored-by: Aolin <[email protected]>
Co-authored-by: Aolin <[email protected]>
ti-chi-bot
bot
added
lgtm
and removed
needs-1-more-lgtm
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: qiancai The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
qiancai
added
the
needs-cherry-pick-release-7.5
|
In response to a cherrypick label: new pull request created to branch |
qiancai
added
the
needs-cherry-pick-master
|
In response to a cherrypick label: new pull request created to branch |
Signed-off-by: ti-chi-bot <[email protected]>
First-time contributors' checklist
What is changed, added or deleted? (Required)
Which TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions.
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?