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

[ja] Translated content/ja/docs/tasks/configure-pod-container/migrate-from-psp.md #48969

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

youmeim
Copy link
Contributor

@youmeim youmeim commented Dec 7, 2024

[ja] Translated content/ja/docs/tasks/configure-pod-container/migrate-from-psp.md

Description

recreated from #48952

Translate
content/en/docs/tasks/configure-pod-container/migrate-from-psp.md
into Japanese
content/ja/docs/tasks/configure-pod-container/migrate-from-psp.md

Note

This PR contains several translation proposal for following terms:

original first experimental translation current proposal
Validating Admission Controller 検証型のアドミッションコントローラー Validating Admission Controller
Mutating Admission Controller 改変型のアドミッションコントローラー Mutating Admission Controller
mutating field Podを改変する設定 Podを変更する設定
Pod Security Standard Podセキュリティ基準 Podセキュリティ基準

Also the format fixation is contained to suite the translation style guide.

Discussion summary on kubernetes-docs-ja (without speaker identity)

  • (Validation|Mutating) Admission Controller は固有名詞といえそう --> 今のところ英文表記がよさそう
  • PSSについては、リソースではなくメジャーな概念とも言えない。 --> 新たに訳出しても問題なさそう。
  • 訳語は普及に合わせて決めれば良い。個々の用語の1:1対応の翻訳は、急がない。
  • 他の分野や他のローカリゼーションチームの事例も参考にすると良い。

Issue

Closes: #48887

Robot

/area localization
/language ja

youmeim and others added 6 commits December 5, 2024 18:06
…-from-psp.md

This commit contains several experimental translation for following words:

Validating Admission Controller: (Pod)検証型のアドミッションコントローラー
Mutating Admission Controller: (Pod)改変型のアドミッションコントローラー
Admission controller is mutating: アドミッションコントローラーがPodを改変する
Pod Security Standard: Podセキュリティ基準 (a modification proposal for 「Podセキュリティの標準」)

Note: These words are not officially authorized by any active committers and localization team members.
If these proposal should not be accepted, these words may be simply rewritten in alphabet or カタカナ語 (IMO).

Further discussion needed.
…p.md

To meet the newline rule on translation style guide (https://kubernetes.io/ja/docs/contribute/localization/#style-guide).
For exception, markdown list or numbered list begining with `-` or `N.` are ignored to apply the newline rule.
…n migrate-from-psp.md, fixed format.

Updated content/ja/docs/tasks/configure-pod-container/migrate-from-psp.md:

* to replace `(改変型|検証型)アドミッションコントローラー` with `(Validating|Mutating) Admission Controller`.
* to replace the word `改変` with `変更` to naturally follow the changes above.
* to fix the markdown format to suite the translation style guide.

The removed trial translations of Admission Controller-related terms may be discussed again.
So these proposals are not required for the purpose of the PR, they are removed.
…d Security level

The variaty of translation for `Pod Security level` is eliminated and it is unified to
`Podセキュリティの水準`.
@k8s-ci-robot k8s-ci-robot added area/localization General issues or PRs related to localization language/ja Issues or PRs related to Japanese language labels Dec 7, 2024
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Dec 7, 2024
@k8s-ci-robot k8s-ci-robot requested a review from t-inu December 7, 2024 20:09
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Dec 7, 2024
Copy link

netlify bot commented Dec 7, 2024

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 1a04043
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6779e9ea7f273400088ffc92
😎 Deploy Preview https://deploy-preview-48969--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.

@youmeim
Copy link
Contributor Author

youmeim commented Dec 16, 2024

/cc Okabe-Junya t-inu
メンション遅れておりました。よろしければ一度お目通しください

preview: https://deploy-preview-48969--kubernetes-io-main-staging.netlify.app/ja/docs/tasks/configure-pod-container/migrate-from-psp/

@Okabe-Junya
Copy link
Member

すみません、直近でレビューできるリソースがないので、もう少し待っていただく or auto-cc でreviewerを再割り当てしたいです... 🙇

@youmeim
Copy link
Contributor Author

youmeim commented Dec 17, 2024

@Okabe-Junya 了解です。急ぎませんので、一先ず待たせて下さい :)

autoccの方が皆さんのご都合に合わせやすいようであれば、今後autoccにて対応します。

Copy link
Member

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

訳語の表記に関しては、ページ内で表記揺れしていないことを優先していただけると助かります 🙇

/label tide/merge-method-squash

Deploy preview: https://deploy-preview-48969--kubernetes-io-main-staging.netlify.app/ja/docs/tasks/configure-pod-container/migrate-from-psp/

@@ -0,0 +1,281 @@
---
title: PodSecurityPolicyからビルトインのPodSecurityアドミッションコントローラーに移行する
Copy link
Member

Choose a reason for hiding this comment

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

ページ内での全体的なAdmission / アドミッションの表記揺れが気になりました

Copy link
Member

Choose a reason for hiding this comment

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

ウェブフック / Webhookの表記揺れの解消もお願いします(NG: WebHook)

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe:

Suggested change
title: PodSecurityPolicyからビルトインのPodSecurityアドミッションコントローラーに移行する
title: PodSecurityPolicyからビルトインのPodセキュリティアドミッションコントローラーに移行する

@k8s-ci-robot k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Dec 22, 2024
@Okabe-Junya
Copy link
Member

/assign nasa9084
for approval

@Okabe-Junya
Copy link
Member

Okabe-Junya commented Dec 22, 2024

autoccの方が皆さんのご都合に合わせやすいようであれば、今後autoccにて対応します。

Prowの/auto-ccは、ランダムにレビュアーを1人抽選・追加するものなので、n週間音沙汰がない、あるいは急いでPRを見てもらいたい際にご自身の判断で実行していただきたいです

FYI: https://prow.k8s.io/command-help

また、いずれの行もPodSecurityPolicyを含まないはずです。

PSPアドミッションコントローラーを無効化したら、(また、ロールバックが必要ないと判断するのに十分な時間が経過したら)、PodSecurityPolicyと関連するRoleやClusterRole、ClusterRoleBindingを削除してもよいでしょう。
(他の無関係な権限付与に使われていないことを確認しておいてください)。
Copy link
Member

Choose a reason for hiding this comment

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

ditto

@youmeim
Copy link
Contributor Author

youmeim commented Dec 25, 2024

@Okabe-Junya @nasa9084
レビュー&コメントありがとうございます。
直近忙しく見られていません。ごめんなさい

今夜まとめて見ます🙇

P.S. 見きれませんでした... 。表記ゆれの件については、考慮すべき背景があるように考えており、改めて共有&相談させて下さい。この点を除き、ご指摘の点は反映しておきます。

Co-authored-by: Junya Okabe <[email protected]>
Co-authored-by: nasa9084 <[email protected]>
@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 ask for approval from nasa9084. For more information see the Kubernetes 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

@youmeim
Copy link
Contributor Author

youmeim commented Dec 26, 2024

ひとまず余分な空白と誤植 (複数形のKubernetes用語) を正しました。

#48969 (comment)

この件については、都合上エディタで一括置換して手動でResolveします。ご了承下さい🙇

Copy link
Member

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

年末年始につき確認が遅くなりましたmm

/lgtm
/cc nasa9084

@k8s-ci-robot k8s-ci-robot requested a review from nasa9084 January 4, 2025 16:51
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 4, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 8712d658d563e7a599dde674cdb898d335efdc6f

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 5, 2025
@k8s-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@youmeim
Copy link
Contributor Author

youmeim commented Jan 5, 2025

@Okabe-Junya ご対応ありがとうございます。今年もよろしくお願いいたします。
LGTM頂いたところ申し訳ないのですが、年末までに対応できていなかった更新点が2件あります。

  1. Podセキュリティ基準ー>Podセキュリティ標準 の表記変更
    (ref: [ja] Translated content/ja/docs/tasks/configure-pod-container/migrate-from-psp.md #48969 (comment))

  2. new issue: [ja] Clarify the localization process for kubernetes-specific terminologies (especially for boundary cases) #49285
    「Pod Securityアドミッションコントローラー」「Pod Security Admission」のような表記ゆれは、このイシューに記載の翻訳フロー(に近いもの)を用いることで生じており、表記ゆれを保持することの妥当性も一定程度あると考えています。このPRでは今のところ、表記ゆれそのものの解消はしていません。(強めのリクエスト、ないしは私が見落としているガイドライン条項があるようなら、表記ゆれは速やかに正します。)
    (ref: [ja] Translated content/ja/docs/tasks/configure-pod-container/migrate-from-psp.md #48969 (comment))

Copy link
Contributor

Choose a reason for hiding this comment

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

I wouldn't write “security“ in English within this page, except where it appears within the API kind (PodSecurityPolicy).

この時点で、Pod Security Admissionを強制するための既存Namespaceの更新が完了しています。
新たなNamespaceに対して適切なPodセキュリティプロファイルを確実に適用できるよう、新規にNamespaceを作るための手順とポリシーについても必ず更新しておくべきです。

ラベルのないNamespaceに対するデフォルトの強制、監査、警告レベルを適用するために、Pod Securityアドミッションコントローラーを静的に定義することもできます。
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
ラベルのないNamespaceに対するデフォルトの強制、監査、警告レベルを適用するために、Pod Securityアドミッションコントローラーを静的に定義することもできます
ラベルのないNamespaceに対するデフォルトの強制、監査、警告レベルを適用するために、Podセキュリティアドミッションコントローラーを静的に定義することもできます

?

@Okabe-Junya
Copy link
Member

Thanks,

/hold
(Once #48969 (comment) and #48969 (comment) resolved, feel free to unhold)

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 6, 2025
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. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. language/ja Issues or PRs related to Japanese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ja] Translate docs/tasks/configure-pod-container/migrate-from-psp.md into Japanese
5 participants