-
Notifications
You must be signed in to change notification settings - Fork 63
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
zh,en: update dmctl commands #515
Conversation
Release Version: v2.0.0 | ||
Git Commit Hash: e6ca256257fbe6e744892841537a16eb84469116 | ||
Git Branch: release-2.0 | ||
UTC Build Time: 2020-10-30 07:43:00 | ||
Go Version: go version go1.13 linux/amd64 |
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.
use v2.0.0 since we now have no hash and build time for v2.0.1
en/dmctl-introduction.md
Outdated
@@ -21,16 +21,16 @@ Enter the interactive mode to interact with DM-master: | |||
{{< copyable "shell-regular" >}} | |||
|
|||
```bash | |||
./dmctl -master-addr 172.16.30.14:8261 | |||
./dmctl -master-addr 172.16.30.14:8261,172.16.30.15:8261 |
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.
How about addressing this behavior was introduced in v2.0.1? (if so, do not forget to update other places in this PR) and in most cases, only one endpoint is enough.
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.
You are right. I remove all of them. 😂
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
Co-authored-by: Charlotte Liu <37295236+CharLotteiu@users.noreply.github.com>
Co-authored-by: Charlotte Liu <37295236+CharLotteiu@users.noreply.github.com>
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
Should we merge this PR after v2.0.1 is released? @csuzhangxc @GMHDBJD |
YES, and #516, #517, #518 should be merged after v2.0.1 released. |
@csuzhangxc, @CharLotteiu, @yikeke, @ti-srebot, PTAL. |
@csuzhangxc, @CharLotteiu, @yikeke, PTAL. |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-2.0 in PR #531 |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com> Co-authored-by: GMHDBJD <35025882+GMHDBJD@users.noreply.github.com>
What is changed, added, or deleted? (Required)
update dmctl command for v2.0.1
Which DM version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
pingcap/dm#1333
pingcap/dm#1316
pingcap/dm#1323
Do your changes match any of the following descriptions?