Skip to content
This repository has been archived by the owner on Jan 3, 2025. It is now read-only.

zh: add document about creating task and verify #77

Merged
merged 17 commits into from
May 22, 2020

Conversation

WangXiangUSTC
Copy link
Contributor

@WangXiangUSTC WangXiangUSTC commented May 18, 2020

What is changed, added or deleted? (Required)

add document about creating task and verify, this document's content is split from zh/deploy-a-dm-cluster-using-binary.md

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

  • master (the latest development version, including v2.0 changes for now)
  • v1.0 (TiDB DM 1.0 versions)

@WangXiangUSTC WangXiangUSTC added the special-week PR from Document Special Week. label May 18, 2020
@yikeke yikeke requested review from lilin90, IANTHEREAL and kissmydb May 19, 2020 02:54
Copy link
Member

@csuzhangxc csuzhangxc left a comment

Choose a reason for hiding this comment

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

rest LGTM

»
```

{{< copyable "" >}}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{{< copyable "" >}}
{{< copyable "shell-regular" >}}

Copy link
Contributor

Choose a reason for hiding this comment

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

The prefix should be », not $. So {{< copyable "" >}} is correct.


## 配置 MySQL 数据源

运行数据同步任务前,需要对 source 进行配置,也就是 MySQL 的相关设置。且为了安全,强制用户配置及使用加密后的密码。首先使用 dmctl 对 MySQL 的密码进行加密,以密码为 "123456" 为例:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
运行数据同步任务前,需要对 source 进行配置,也就是 MySQL 的相关设置。且为了安全,强制用户配置及使用加密后的密码。首先使用 dmctl 对 MySQL 的密码进行加密,以密码为 "123456" 为例:
运行数据同步任务前,需要对 source 进行配置,也就是 MySQL 的相关设置。且为了安全,必须配置及使用加密后的密码。首先使用 dmctl 对 MySQL 的密码进行加密,以密码为 "123456" 为例:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个后面就不是强制的了, 改成了 建议配置及使用加密后的密码

@WangXiangUSTC WangXiangUSTC added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 19, 2020
@toutdesuite toutdesuite added the translation/welcome Waits for a contributor to translate this PR to docs-dm/zh or docs-dm/en. label May 19, 2020
@WangXiangUSTC WangXiangUSTC added status/PTAL This PR is ready for reviewing. and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels May 19, 2020
WangXiangUSTC and others added 3 commits May 19, 2020 17:59
Copy link
Member

@csuzhangxc csuzhangxc left a comment

Choose a reason for hiding this comment

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

LGTM

@csuzhangxc csuzhangxc added status/LGT1 Indicates that a PR has LGTM 1. and removed status/PTAL This PR is ready for reviewing. labels May 19, 2020
@yikeke yikeke removed their request for review May 20, 2020 04:41
@yikeke
Copy link
Contributor

yikeke commented May 20, 2020

Please add this file link into TOC.md. @WangXiangUSTC

@WangXiangUSTC
Copy link
Contributor Author

Please add this file link into TOC.md. @WangXiangUSTC

ok, already updated @yikeke

@sre-bot sre-bot added the status/PTAL This PR is ready for reviewing. label May 21, 2020
@sre-bot sre-bot requested a review from lilin90 May 21, 2020 03:44
@yikeke yikeke requested review from miaoqingli and removed request for kissmydb May 21, 2020 03:56
fCxfQ9XKCezSzuCD0Wf5dUD+LsKegSg=
```

记录该加密后的密码,用于下面 MySQL1 的配置。把以下配置文件内容写入到 `conf/source1.toml` 中。

Choose a reason for hiding this comment

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

Suggested change
记录该加密后的密码,用于下面 MySQL1 的配置。把以下配置文件内容写入到 `conf/source1.toml` 中。
记录该加密后的密码,可以用于新增数据源。比如下面这个例子,可以新建一个文件,把以下配置文件内容写入到 `conf/source1.toml` 中。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

现在文件内容不太一样,我做了下调整

./bin/dmctl --master-addr=192.168.0.4:8261 operate-source create conf/source1.toml
```

对于 MySQL2,修改配置文件中的 `name` 为 `mysql-replica-02`,`host` 为 `192.168.0.2`,并将 `password` 和 `port` 改为相应的值即可。
Copy link

@miaoqingli miaoqingli May 21, 2020

Choose a reason for hiding this comment

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

Suggested change
对于 MySQL2,修改配置文件中的 `name``mysql-replica-02``host``192.168.0.2`,并将 `password``port` 改为相应的值即可
如果要再新增一个 MySQL2 数据源,可以修改配置文件中的 `name``mysql-replica-02``host``192.168.0.2`,并将 `password``port` 改为相应的值

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改了

@toutdesuite
Copy link
Contributor

@miaoqingli PTAL again, thanks

@miaoqingli
Copy link

LGTM

@WangXiangUSTC WangXiangUSTC added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels May 22, 2020
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Copy link
Contributor

@toutdesuite toutdesuite 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 toutdesuite added the status/can-merge Indicates a PR has been approved by a committer. label May 22, 2020
@sre-bot
Copy link

sre-bot commented May 22, 2020

/run-all-tests

@sre-bot sre-bot merged commit c60d215 into docs-special-week May 22, 2020
@WangXiangUSTC WangXiangUSTC deleted the xiang/deploy branch May 22, 2020 07:14
lilin90 added a commit that referenced this pull request May 27, 2020
* scripts: add a script to run markdownlint check locally (#74)

* Update ci.yaml (#84)

* zh: update TOC (#75)

* update TOC

* Update TOC.md

* Update TOC.md

* Update zh/TOC.md

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

* Update zh/TOC.md

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

* address comments

* fix dead link and address comment

* Apply suggestions from code review

* fix levels

* Update links in TOC

* fix CI

* remove link for 测试验证

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

* zh: add the requirement of software and hardware (#76)

* add requirement

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* update word

* address comemnt

* Apply suggestions from code review

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* update disk requirement

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* rename file name

* Apply suggestions from code review

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* address comment

Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* zh: update documents about error handling (#78)

* zh: add a document about performance handling (#81)

* zh: add document about creating task and verify (#77)

* zh: add quick start doc for dm deployment (#80)

* zh: update docs about key features (#86)

* zh: add a document about how to optimize config (#89)

* add config optimize

* rename file

* Apply suggestions from code review

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>

* format

* address comment

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/tune-configuration.md

* Apply suggestions from code review

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/tune-configuration.md

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>

* zh: Further update TOC.md (#92)

* Update TOC.md

* add omitted links

* update format

* Update TOC.md

* Update TOC.md

* Update zh/TOC.md

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

* update file name

* add alias

* fix dead links

* Revert "fix dead links"

This reverts commit 9cf7b75.

* fix dead links in zh

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

* zh: update dm-upgrade (#87)

* zh: add a document about scenarios (#88)

* zh: update daily-check.md (#91)

* update daily-check.md

* address comment

* address comment

* Update zh/daily-check.md

* Update zh/daily-check.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/daily-check.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/daily-check.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: WangXiangUSTC <wx347249478@gmail.com>

* zh: update docs about alert rules and handling (#93)

* zh: add a doc about command-line-flags and update TOC (#94)

* zh: add a doc about command-line-flags and update TOC

* zh: update TOC

* command-line-flags: address comments

* Update zh/command-line-flags.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/command-line-flags.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* Update zh/command-line-flags.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* command-line-flags: address comments

* remove empty files

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* zh: add files about manage task (#90)

* add task manage for special week

* minor change

* rename file

* rename file

* minor change

* minor change

* fix CI

* fix CI

* address comment

* minor change

* Update skip-or-replace-abnormal-sql-statements.md

* Update zh/create-task.md

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>

* address comment

* address comment

* update TOC

* address comment

* Update TOC.md

* add query-status and query-error in TOC

* fix toc

* Update zh/query-error.md

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* Update zh/query-status.md

Co-authored-by: Lilian Lee <lilin@pingcap.com>

* rm manage-replication-tasks.md

* fix CI

* fix CI

* address comment

* fix bug

* update TOC

* add dmctl-intro.md

* Update zh/dmctl-intro.md

Co-authored-by: toutdesuite <guizhiluo2014@163.com>

* rename file

* Update TOC.md

* address comment

Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>
Co-authored-by: WangXiangUSTC <wx347249478@gmail.com>

* zh: add document about how to do performance test (#85)

* zh: update TOC and remove empty files (#95)

* update TOC and remove empty files

* Update TOC.md

Co-authored-by: yikeke <yikeke@pingcap.com>

* zh: update release notes in toc

* zh: update wording

* *: make image name unique at the repo level (#96)

* *: make image name unique at repo level

* en: fix one link

Co-authored-by: Ran <huangran@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: toutdesuite <guizhiluo2014@163.com>
Co-authored-by: yikeke <yikeke@pingcap.com>
Co-authored-by: WangXiangUSTC <wx347249478@gmail.com>
Co-authored-by: Xuecheng Zhang <csuzhangxc@gmail.com>
Co-authored-by: Chunzhu Li <lichunzhu@stu.xjtu.edu.cn>
Co-authored-by: GMHDBJD <35025882+GMHDBJD@users.noreply.github.com>
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@toutdesuite toutdesuite added translation/doing This PR’s assignee is translating this PR. and removed translation/welcome Waits for a contributor to translate this PR to docs-dm/zh or docs-dm/en. labels Jun 18, 2020
@toutdesuite toutdesuite assigned toutdesuite and unassigned TomShawn Jun 24, 2020
@toutdesuite toutdesuite added translation/done This PR has been translated. and removed translation/doing This PR’s assignee is translating this PR. labels Jun 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
special-week PR from Document Special Week. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. status/PTAL This PR is ready for reviewing. translation/done This PR has been translated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants