Skip to content

[ja] Translate content/en/docs/reference/glossary/taint.md into Japanese #50399

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yuto-kimura-g
Copy link
Contributor

Description

Translate content/en/docs/reference/glossary/taint.md into Japanese

Issue

Closes: #50369

FYI

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign nasa9084 for approval. For more information see the Code Review Process.

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 cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. area/localization General issues or PRs related to localization language/ja Issues or PRs related to Japanese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 5, 2025
Copy link

netlify bot commented Apr 5, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit d3b4218
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/67f140d65f277f00084a1c81
😎 Deploy Preview https://deploy-preview-50399--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@t-inu
Copy link
Member

t-inu commented Apr 20, 2025

/cc

@k8s-ci-robot k8s-ci-robot requested a review from t-inu April 20, 2025 06:40
date: 2019-01-11
full_link: /ja/docs/concepts/scheduling-eviction/taint-and-toleration/
short_description: >
key、value、effectの3つの必須属性からなり、Podが特定のノードやノードグループにスケジューリングされることを防ぎます。
Copy link
Member

Choose a reason for hiding this comment

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

コアオブジェクトではないため、#45972core-objectタグを外す対処がされたとのことですが、原文の説明には残っているのですね。

A core object consisting of three required properties: key, value, and effect.

それを意識して、コアオブジェクトは省いて訳されたのだと思われますが、原文を修正する提案をしたほうがよさそうですね。

あと気になるのは、"required properties"と書かれていますが、valueは任意のはずなので、requiredと表現する必要があるのかなという点です。
これは、Tolerationの説明も同様です。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@t-inu

レビューありがとうございます。

原文の説明には残っているのですね。

はい、 50050df のコミットでこのような(core object、required properties)説明になったようです。

あと気になるのは、"required properties"と書かれていますが、valueは任意のはずなので、requiredと表現する必要があるのかなという点です。

私も、requiredと表現する必要は無いと思います。Taintであればご指摘の通りvalueは任意ですし、Tolerationであればkey, value, effectのいずれかを省略することなども可能です。
しかし正確に説明すると長くなるので、上記コミット 50050df で修正前のA key-value pair and an effectという表現で良いのではないかと思います。Tolerationについてはoperatorもあるので別の表現を考えても良さそうですが。

原文を修正する提案をしたほうがよさそうですね。

上記コミットの一部(TaintのL7, L14)を元に戻し、Tolerationのrequiredという言い回しを変更するissue, PRを新しく出す感じで良いでしょうか?
ご教示ください。

Copy link
Member

Choose a reason for hiding this comment

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

なるほど、この説明文は一度手が入っていたのですね。
読みにくいから改善したという経緯なので、元に戻す提案は受け入れられにくいかもしれないです。

Taints prevent the scheduling of Pods on nodes or node groups using three properties: key, value, and effect.
みたいなのでどうでしょうかね。
Tolerationのほうは、まずはrequiredを取り除くだけで反応を伺ってもいいかもです。

issueとしては、

  • Taintはコアオブジェクトではないためタグを外したにもかかわらず、説明文にコアオブジェクトと書かれている。
  • TaintもTolerationも、少なくともvalueはoptionalなので、requiredという表現は不要ではないか。

という点に言及し、着手して問題なさそうであれば、PRを作成という流れで進めてもらえると助かります。
私もウォッチしていきます。

@Okabe-Junya
Copy link
Member

/assign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/localization General issues or PRs related to localization cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/ja Issues or PRs related to Japanese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ja] Translate content/en/docs/reference/glossary/taint.md into Japanese
4 participants