Skip to content

Conversation

@JunichiroKohari
Copy link
Contributor

@JunichiroKohari JunichiroKohari commented Dec 1, 2025

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. area/localization General issues or PRs related to localization labels Dec 1, 2025
@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 inductor 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 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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Dec 1, 2025
@netlify
Copy link

netlify bot commented Dec 1, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 9df353f
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69358e199729270008137a70
😎 Deploy Preview https://deploy-preview-53496--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 project configuration.

@JunichiroKohari JunichiroKohari marked this pull request as ready for review December 1, 2025 01:09
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 1, 2025
Copy link
Member

@kfess kfess left a comment

Choose a reason for hiding this comment

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

コントリビューションありがとうございます!
複数点について、レビューさせていただきました。

JunichiroKohari and others added 8 commits December 2, 2025 19:51
Add section link

Co-authored-by: kfess <[email protected]>
translate glossary text to ja

Co-authored-by: kfess <[email protected]>
apply translation guideline rule

Co-authored-by: kfess <[email protected]>
improve translation

Co-authored-by: kfess <[email protected]>
static Pod

Co-authored-by: kfess <[email protected]>
クラスタ to クラスター

Co-authored-by: kfess <[email protected]>
@JunichiroKohari
Copy link
Contributor Author

@kfess
ご確認ありがとうございます!
ガイドラインに則っていない箇所があり失礼しました。
ご提案いただいたsuggestionを取り入れ、空行の追加も行いました。

Copy link
Member

@kfess kfess left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます!
追加で、もう数点コメントさせていただきました。
ほぼLGTMです。

指定されていない場合、保存時の暗号化は有効になりません。

この設定ファイルはYAML形式で、[`EncryptionConfiguration`](/docs/reference/config-api/apiserver-config.v1/)という種類の設定APIを表します。
設定の例は[Encryption at rest configuration](/docs/tasks/administer-cluster/encrypt-data/#understanding-the-encryption-at-rest-configuration)で確認できます。
Copy link
Member

Choose a reason for hiding this comment

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

リンク先の日本語訳が存在しない場合でも、翻訳して良いという判断のため、コチラも適切な翻訳をお願いできますでしょうか。

ref: #51318 (comment)

Copy link
Member

Choose a reason for hiding this comment

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

下記のように訳するのはいかがでしょうか?

Suggested change
設定の例は[Encryption at rest configuration](/docs/tasks/administer-cluster/encrypt-data/#understanding-the-encryption-at-rest-configuration)で確認できます。
設定の例は[保存時の暗号化設定](/docs/tasks/administer-cluster/encrypt-data/#understanding-the-encryption-at-rest-configuration)で確認できます。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます!ご提案を取り入れました。

Copy link
Contributor Author

@JunichiroKohari JunichiroKohari Dec 8, 2025

Choose a reason for hiding this comment

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

保存時の暗号化設定

こちらは、issue #51635 が作られており、assigneeもいるためその方が翻訳をしてくださると思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kfess

EncryptionConfiguration

こちらはissueがなく対応者がいないため私が翻訳しようかと思うのですが、その場合issueを作るところから始めれば良いのでしょうか?

Copy link
Member

@kfess kfess Dec 8, 2025

Choose a reason for hiding this comment

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

@JunichiroKohari

是非、翻訳いただけると助かります。ただ、reference 系のページは、レビューが少し低優先度になってしまうかもしれません。
ref: #53391 (comment)

もし、他のページの翻訳についてもご協力いただけるのであれば、
翻訳状況可視化ツール で、未翻訳のページなどについて対応いただけると嬉しいです!

@kfess
Copy link
Member

kfess commented Dec 7, 2025

ありがとうございます!
/lgtm

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

LGTM label has been added.

Git tree hash: 9b397066a5df921b1c262dc46e215fa85eed70b3

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 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/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ja] Translate content/en/docs/tasks/administer-cluster/decrypt-data.md into Japanese

3 participants