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

Update safely-drain-node.md #25996

Closed
wants to merge 1 commit into from
Closed

Conversation

hantmac
Copy link
Contributor

@hantmac hantmac commented Jan 7, 2021

fix typo

fix typo

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 7, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign tanjunchen after the PR has been reviewed.
You can assign the PR to them by writing /assign @tanjunchen in a comment when ready.

The full list of commands accepted by this bot can be found 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 language/zh Issues or PRs related to Chinese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 7, 2021
@netlify
Copy link

netlify bot commented Jan 7, 2021

✔️ Deploy preview for kubernetes-io-master-staging ready!

🔨 Explore the source changes: a4702eb

🔍 Inspect the deploy logs: https://app.netlify.com/sites/kubernetes-io-master-staging/deploys/5ff6d6d858131b00078d9b3c

😎 Browse the preview: https://deploy-preview-25996--kubernetes-io-master-staging.netlify.app

@@ -127,7 +127,7 @@ Once it returns (without giving an error), you can power down the node
If you leave the node in the cluster during the maintenance operation, you need to run
-->
一旦它返回(没有报错),
你就可以下电此节点(或者等价地,如果在云平台上,删除支持该节点的虚拟机)。
你就可以下掉此节点(或者等价地,如果在云平台上,删除支持该节点的虚拟机)。
Copy link
Contributor

Choose a reason for hiding this comment

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

emm ... 原来的翻译就是”下电“,power down。
如果觉得不合适,可以换一种说法。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

感觉直译很难理解呀,其实 power down 对于中文来说就是下线节点的意思吧

Copy link
Contributor

Choose a reason for hiding this comment

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

... 难道不能说关掉节点电源?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

可以呀,其实这些操作的描述,可以统称为 下线 吧?不管你是关闭电源,还是直接回收节点,对于 k8s 集群和我们集群管理员来说,都是下线这台节点。

Copy link
Contributor

Choose a reason for hiding this comment

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

这是个见仁见智的问题。个人认为 “下掉” 并不比现在的 “下电” 更好。

Copy link
Contributor

Choose a reason for hiding this comment

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

我认为 下线 比较合适。

Copy link
Member

Choose a reason for hiding this comment

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

剛剛參考了一下英文版的內容,內文如下:
Once it returns (without giving an error), you can power down the node (or equivalently, if on a cloud platform, delete the virtual machine backing the node). If you leave the node in the cluster during the maintenance operation, you need to run.

認為根因是power down這個詞,
本身描述的太過於抽象了。
無法直覺聯想,該文章想要是描述Linux系統$ shutdown$ halt或是$ poweroff
還是根據下段句子,是描述K8s的$ kubectl cordon

如果是$ poweroff,應該就是 “下电”這個描述好。
反之是$ shutdown,應該就是 “下线”這個描述好。

Copy link
Contributor

Choose a reason for hiding this comment

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

嗯,“下线”似乎更常见。

Copy link
Member

@ydFu ydFu Jan 25, 2021

Choose a reason for hiding this comment

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

@hantmac
I recommend the typo to be translated "下线"

@tengqm
Copy link
Contributor

tengqm commented Feb 6, 2021

/close

@k8s-ci-robot
Copy link
Contributor

@tengqm: Closed this PR.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ydFu
Copy link
Member

ydFu commented Feb 6, 2021

@tengqm
If this still needs adjustment, I can compress it in my next translation PR.

@tengqm
Copy link
Contributor

tengqm commented Feb 6, 2021

@ydFu 用 "下线" 或许好一些。

@ydFu
Copy link
Member

ydFu commented Feb 6, 2021

@tengqm
Glad to know we’re all on the same page,
I also think "下线" be better than others.
and I adjust this update in PR #26395.

ydFu added a commit to ydFu/website that referenced this pull request Feb 19, 2021
* Resync with english version in 'Dual-stack docs correction kubernetes#26386'

* Update in 'administer-cluster\safely-drain-node.md' kubernetes#25996

* Fix rebase in services-networking\dual-stack.md

Signed-off-by: ydFu <[email protected]>
ydFu added a commit to ydFu/website that referenced this pull request Feb 19, 2021
* Resync with english version in 'Dual-stack docs correction kubernetes#26386'

* Update in 'administer-cluster\safely-drain-node.md' kubernetes#25996

* Fix rebase in services-networking\dual-stack.md.

Signed-off-by: ydFu <[email protected]>
ydFu added a commit to ydFu/website that referenced this pull request Feb 19, 2021
* Resync with english version in 'Dual-stack docs correction kubernetes#26386'

* Update in 'administer-cluster\safely-drain-node.md' kubernetes#25996

* Fix git rebase in services-networking\dual-stack.md

Signed-off-by: ydFu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/zh Issues or PRs related to Chinese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants