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

[zh] translation for the run-app section #29451

Merged

Conversation

steven-my
Copy link
Contributor

@steven-my steven-my commented Aug 18, 2021

related: #29329
Translation for the run-app section in issue 29329.
Modified:
content/zh/docs/tasks/debug-application-cluster/audit.md
content/zh/docs/tasks/debug-application-cluster/debug-cluster.md
content/zh/docs/tasks/debug-application-cluster/debug-running-pod.md
content/zh/docs/tasks/debug-application-cluster/determine-reason-pod-failure.md
content/zh/docs/tasks/run-application/delete-stateful-set.md
content/zh/docs/tasks/run-application/horizontal-pod-autoscale.md

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Aug 18, 2021
@k8s-ci-robot k8s-ci-robot added language/zh Issues or PRs related to Chinese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 18, 2021
@netlify
Copy link

netlify bot commented Aug 18, 2021

✔️ Deploy Preview for kubernetes-io-main-staging ready!

🔨 Explore the source changes: 742e7d7

🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/61231f68f1d358000860e022

😎 Browse the preview: https://deploy-preview-29451--kubernetes-io-main-staging.netlify.app

@tengqm
Copy link
Contributor

tengqm commented Aug 20, 2021

@steven-my Please remove 'Fixes #29329' from your commit message and recommit this PR. This PR is related to #29329 but it is NOT supposed to close it. You can use related: #29329 to link this PR to the umbrella issue.

@@ -234,7 +231,7 @@ You can view the state of the newly created ephemeral container using `kubectl d

{{< note >}}
{{< glossary_tooltip text="容器运行时" term_id="container-runtime" >}}必须支持`--target`参数。
如果不支持,则临时容器可能不会启动,或者可能使用隔离的进程命名空间启动
如果不支持,则临时容器可能不会启动,或者可能使用隔离的进程命名空间启动以便 "ps" 不在其他容器内显示进程
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
如果不支持,则临时容器可能不会启动,或者可能使用隔离的进程命名空间启动以便 "ps" 不在其他容器内显示进程。
如果不支持,则临时容器可能不会启动,或者可能使用隔离的进程命名空间启动,
以便 "ps" 不显示其他容器内的进程。

滚动更新,它们会为你管理底层副本数。
Pod 水平扩缩只支持后一种:HPA 会被绑定到 Deployment 对象,
HPA 设置副本数量时,Deployment 会设置底层副本数。
Kubernetes 允许你在 Deployment 时执行滚动更新。在这种情况下,由 Deployment 为你管理下层的 ReplicaSet。
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
Kubernetes 允许你在 Deployment 时执行滚动更新。在这种情况下,Deployment 为你管理下层的 ReplicaSet。
Kubernetes 允许你在 Deployment 上执行滚动更新。在这种情况下,Deployment 为你管理下层的 ReplicaSet。

Pod 水平扩缩只支持后一种:HPA 会被绑定到 Deployment 对象,
HPA 设置副本数量时,Deployment 会设置底层副本数。
Kubernetes 允许你在 Deployment 时执行滚动更新。在这种情况下,由 Deployment 为你管理下层的 ReplicaSet。
当你为一个 Deployment 配置扩缩时,你为单个 Deployment 绑定一个 HorizontalPodAutoscaler。
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
当你为一个 Deployment 配置扩缩时,你为单个 Deployment 绑定一个 HorizontalPodAutoscaler。
当你为一个 Deployment 配置自动扩缩时,你要为每个 Deployment 绑定一个 HorizontalPodAutoscaler。

HPA 设置副本数量时,Deployment 会设置底层副本数。
Kubernetes 允许你在 Deployment 时执行滚动更新。在这种情况下,由 Deployment 为你管理下层的 ReplicaSet。
当你为一个 Deployment 配置扩缩时,你为单个 Deployment 绑定一个 HorizontalPodAutoscaler。
HorizontalPodAutoscaler 管理 Deployment 的 “replicas” 字段。
Copy link
Contributor

Choose a reason for hiding this comment

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

在 markdown 中,反引号是用来以等宽字体显示变量或代码的,不要改成双引号!

Suggested change
HorizontalPodAutoscaler 管理 Deployment 的 replicas 字段。
HorizontalPodAutoscaler 管理 Deployment 的 `replicas` 字段。

Kubernetes 允许你在 Deployment 时执行滚动更新。在这种情况下,由 Deployment 为你管理下层的 ReplicaSet。
当你为一个 Deployment 配置扩缩时,你为单个 Deployment 绑定一个 HorizontalPodAutoscaler。
HorizontalPodAutoscaler 管理 Deployment 的 “replicas” 字段。
Deployment Controller 负责设置下层 ReplicaSet 的 “replicas”字段, 以便于在滚动及后续过程中为其增加到适当的数量
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
Deployment Controller 负责设置下层 ReplicaSet 的 replicas字段, 以便于在滚动及后续过程中为其增加到适当的数量。
Deployment Controller 负责设置下层 ReplicaSet 的 `replicas`字段,
以便确保在上线及后续过程副本个数合适。

通过直接操控副本控制器执行滚动升级时,HPA 不能工作,
也就是说你不能将 HPA 绑定到某个 RC 再执行滚动升级。
HPA 不能工作的原因是它无法绑定到滚动更新时所新创建的副本控制器。
如果你对一个 replicas 值被自动扩缩过的 StatefulSet 执行滚动更新, 这个 StatefulSet 会直接管理他的 Pod 集合 (不存在类似 ReplicaSet 的中间资源)。
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
如果你对一个 replicas 值被自动扩缩过的 StatefulSet 执行滚动更新, 这个 StatefulSet 会直接管理他的 Pod 集合 (不存在类似 ReplicaSet 的中间资源)。
如果你对一个副本个数被自动扩缩的 StatefulSet 执行滚动更新, StatefulSet
会直接管理它的 Pod 集合 (不存在类似 ReplicaSet 这样的中间资源)。

@steven-my steven-my force-pushed the 29329-translation-for-run-app branch from 3637ba6 to a0d7704 Compare August 23, 2021 03:57
@steven-my steven-my force-pushed the 29329-translation-for-run-app branch from a0d7704 to 742e7d7 Compare August 23, 2021 04:09
@steven-my
Copy link
Contributor Author

Hi @tengqm , thanks for review my changes. I modified files as your comments.

@reylejano
Copy link
Member

/retitle [zh] translation for the run-app section

@k8s-ci-robot k8s-ci-robot changed the title translation for the run-app section [zh] translation for the run-app section Aug 24, 2021
@tengqm
Copy link
Contributor

tengqm commented Aug 25, 2021

Thanks.
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tengqm

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 25, 2021
@howieyuen
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 31, 2021
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 8ac91a13e9bd61d570700bde1e8eaa90844233df

@k8s-ci-robot k8s-ci-robot merged commit 82da8f9 into kubernetes:main Aug 31, 2021
@tengqm tengqm mentioned this pull request Sep 26, 2021
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/zh Issues or PRs related to Chinese language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants