-
Notifications
You must be signed in to change notification settings - Fork 120
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, zh: update documentation about PodRestarter #724
Conversation
Signed-off-by: Lonng <[email protected]>
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
@@ -85,13 +85,3 @@ To activate the graceful logoff feature, you need to enable some related configu | |||
2. Install or update TiDB Operator. | |||
|
|||
To install or update TiDB Operator, see [Deploy TiDB Operator in Kubernetes](deploy-tidb-operator.md). | |||
|
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 remove the whole section of https://docs.pingcap.com/tidb-in-kubernetes/dev/restart-a-tidb-cluster#gracefully-restart-a-single-pod-of-the-tidb-component
en/restart-a-tidb-cluster.md
Outdated
@@ -36,23 +36,23 @@ If you find that the memory leak occurs in a Pod during use, you need to restart | |||
storage: "1Gi" | |||
config: {} | |||
annotations: | |||
tidb.pingcap.com/restartedAt: "202004201200" | |||
tidb.pingcap.com/restartedAt: 2020-04-20 12:00 |
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.
tidb.pingcap.com/restartedAt: 2020-04-20 12:00 | |
tidb.pingcap.com/restartedAt: 2020-04-20-12-00 |
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.
这样可以吗?格式一致一些
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.
在 571115b 中改成了 rfc3339 的格式。
zh/restart-a-tidb-cluster.md
Outdated
@@ -32,23 +32,23 @@ aliases: ['/docs-cn/tidb-in-kubernetes/dev/restart-a-tidb-cluster/'] | |||
storage: "1Gi" | |||
config: {} | |||
annotations: | |||
tidb.pingcap.com/restartedAt: "202004201200" | |||
tidb.pingcap.com/restartedAt: 2020-04-20 12:00 |
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.
comments 同上,还有下面的重启单个 Pod 部分全部删掉。
@lonng, please update your pull request. |
2 similar comments
@lonng, please update your pull request. |
@lonng, please update your pull request. |
No updates for a long time, close PR. |
@lonng Please address comments. Thanks~ |
Signed-off-by: Lonng <[email protected]>
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
Signed-off-by: ti-srebot <[email protected]>
cherry pick to release-1.1 in PR #779 |
Signed-off-by: ti-srebot <[email protected]> Co-authored-by: Lonng <[email protected]>
Signed-off-by: Lonng [email protected]
What is changed, added or deleted? (Required)
Update documentation about PodRestarter.
Which TiDB Operator version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
pingcap/tidb-operator#3296
Do your changes match any of the following descriptions?