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

dev, v3.0, v2.1: add DM upgrade advice #1656

Merged
merged 5 commits into from
Aug 13, 2019
Merged

Conversation

3pointer
Copy link
Contributor

@3pointer 3pointer commented Jul 24, 2019

What is changed, added or deleted?

Add incompatible DM upgrade version advice after this PR is merged.

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

pingcap/dm#212

Checklist

  • No Tab spaces anywhere
  • Leave a blank line both before and after a code block
  • Keep the first level heading consistent with title in metadata
  • Use four spaces for each level of indentation except that in TOC.md

@CLAassistant
Copy link

CLAassistant commented Jul 24, 2019

CLA assistant check
All committers have signed the CLA.

启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
可能遗留的废弃配置:

- dm-worker.toml中的 `meta-file`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- dm-worker.toml中的 `meta-file`
- dm-worker.toml 中的 `meta-file`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

可能遗留的废弃配置:

- dm-worker.toml中的 `meta-file`
- task.yaml中的 `mysql-instances` 中的 `server-id`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- task.yaml中的 `mysql-instances` 中的 `server-id`
- task.yaml 中的 `mysql-instances` 中的 `server-id`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@amyangfei amyangfei left a comment

Choose a reason for hiding this comment

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

LGTM

@amyangfei
Copy link
Contributor

PTAL @CaitinChen

@yikeke yikeke requested review from lilin90 and CaitinChen July 26, 2019 10:51
@lilin90 lilin90 changed the title dm upgrade advice dev, v3.0, v2.1: add DM upgrade advice Aug 5, 2019

### 主要变更

从此版本开始,所有的配置进行严格的检查,遇到不识别的配置会报错,这样的目的是为了确保用户始终准确的了解自己的配置。
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
从此版本开始,所有的配置进行严格的检查,遇到不识别的配置会报错,这样的目的是为了确保用户始终准确的了解自己的配置
从此版本开始,将对所有的配置进行严格检查,遇到不识别的配置会报错,以确保用户始终准确地了解自己的配置


### 升级操作示例

启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
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
启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
启动 DM-master 或 DM-worker 前,必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。


### 主要变更

从此版本开始,所有的配置进行严格的检查,遇到不识别的配置会报错,这样的目的是为了确保用户始终准确的了解自己的配置。
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
从此版本开始,所有的配置进行严格的检查,遇到不识别的配置会报错,这样的目的是为了确保用户始终准确的了解自己的配置
从此版本开始,将对所有的配置进行严格检查,遇到不识别的配置会报错,以确保用户始终准确地了解自己的配置


### 主要变更

从此版本开始,所有的配置进行严格的检查,遇到不识别的配置会报错,这样的目的是为了确保用户始终准确的了解自己的配置。
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
从此版本开始,所有的配置进行严格的检查,遇到不识别的配置会报错,这样的目的是为了确保用户始终准确的了解自己的配置
从此版本开始,将对所有的配置进行严格检查,遇到不识别的配置会报错,以确保用户始终准确地了解自己的配置


### 升级操作示例

启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
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
启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
启动 DM-master 或 DM-worker 前,必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。

可能遗留的废弃配置:

- dm-worker.toml 中的 `meta-file`
- task.yaml 中的 `mysql-instances` 中的 `server-id`
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
- task.yaml 中的 `mysql-instances` 中的 `server-id`
- `task.yaml` 中的 `mysql-instances` 中的 `server-id`

启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
可能遗留的废弃配置:

- dm-worker.toml 中的 `meta-file`
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
- dm-worker.toml 中的 `meta-file`
- `dm-worker.toml` 中的 `meta-file`

可能遗留的废弃配置:

- dm-worker.toml 中的 `meta-file`
- task.yaml 中的 `mysql-instances` 中的 `server-id`
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
- task.yaml 中的 `mysql-instances` 中的 `server-id`
- `task.yaml` 中的 `mysql-instances` 中的 `server-id`

启动 dm-master/dm-worker 前确保废弃的配置信息已经删除,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
可能遗留的废弃配置:

- dm-worker.toml 中的 `meta-file`
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
- dm-worker.toml 中的 `meta-file`
- `dm-worker.toml` 中的 `meta-file`

可能遗留的废弃配置:

- dm-worker.toml 中的 `meta-file`
- task.yaml 中的 `mysql-instances` 中的 `server-id`
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
- task.yaml 中的 `mysql-instances` 中的 `server-id`
- `task.yaml` 中的 `mysql-instances` 中的 `server-id`


### 升级操作示例

启动 DM-master 或 DM-worker 前, 必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
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
启动 DM-master 或 DM-worker 前, 必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
启动 DM-master 或 DM-worker 前必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。


### 升级操作示例

启动 DM-master 或 DM-worker 前, 必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
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
启动 DM-master 或 DM-worker 前, 必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
启动 DM-master 或 DM-worker 前必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。


### 升级操作示例

启动 DM-master 或 DM-worker 前,必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
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
启动 DM-master 或 DM-worker 前,必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。
启动 DM-master 或 DM-worker 前必须确保已经删除废弃的配置信息,且没有多余的配置项,否则会启动失败。可根据失败信息删除多余的配置。

@lilin90 lilin90 added the translation/doing This PR’s assignee is translating this PR. label Aug 7, 2019
@CaitinChen CaitinChen removed their request for review August 9, 2019 07:13
@lilin90
Copy link
Member

lilin90 commented Aug 12, 2019

@3pointer Please address comments. Thanks!

@3pointer
Copy link
Contributor Author

@3pointer Please address comments. Thanks!

Done

Copy link
Member

@lilin90 lilin90 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 lilin90 merged commit bebceb4 into pingcap:master Aug 13, 2019
@TomShawn TomShawn added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Aug 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants