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

[ko] Update outdated files in dev-1.26.-ko.1 [M18-22] #40988

Conversation

bugoverdose
Copy link
Member

from #38458

I have updated some of the outdated contents in dev-1.26-ko.1 branch.
You can check the full comparison here.

Updated Files

  • M18. content/en/docs/concepts/configuration/secret.md | 64(+M) 123(-)
  • M19. content/en/docs/concepts/containers/_index.md | 18(+S) 10(-)
  • M20. content/en/docs/concepts/containers/container-lifecycle-hooks.md | 1(+XS) 1(-)
  • M21. content/en/docs/concepts/containers/images.md | 22(+S) 0(-)
  • M22. content/en/docs/concepts/containers/runtime-class.md | 2(+XS) 1(-)

@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. language/ko Issues or PRs related to Korean language labels May 6, 2023
@k8s-ci-robot k8s-ci-robot requested review from ianychoi and jmyung May 6, 2023 07:48
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label May 6, 2023
@netlify
Copy link

netlify bot commented May 6, 2023

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 4135336
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/647039ed3cc3240008837475
😎 Deploy Preview https://deploy-preview-40988--k8s-dev-ko.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 settings.

@ysyukr
Copy link
Member

ysyukr commented May 11, 2023

/assign

@ysyukr
Copy link
Member

ysyukr commented May 22, 2023

@bugoverdose 리뷰가 늦었습니다.
대부분의 파일들이 영문문서와 라인이 일치하지 않습니다.
줄바꿈 등을 통해 영문문서와 라인수를 맞춰주시기를 부탁드리겠습니다.
라인수가 동일해야, 지금뿐만 아니라, 추후 해당문서 유지보수시에 빠르게 처리할수 있기 때문입니다.

@bugoverdose
Copy link
Member Author

@bugoverdose 리뷰가 늦었습니다. 대부분의 파일들이 영문문서와 라인이 일치하지 않습니다. 줄바꿈 등을 통해 영문문서와 라인수를 맞춰주시기를 부탁드리겠습니다. 라인수가 동일해야, 지금뿐만 아니라, 추후 해당문서 유지보수시에 빠르게 처리할수 있기 때문입니다.

문제가 된 파일들 3개 모두 원문과 라인수를 일치시켰습니다.

다만, M21의 159~161줄의 경우 원문과는 별개로 한글화 과정에서 줄바꿈을 추가함에 따라 원문과의 전체 라인수에 불일치가 존재했는데요.

위의 커밋에서 원문에 맞춰 수정했다가, 의도적으로 추가한 것 같아 아래 커밋에서 바로 롤백했습니다.

Copy link
Member

@ysyukr ysyukr left a comment

Choose a reason for hiding this comment

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

@bugoverdose 고생하셨습니다.
images 문서는 다음버전에서 정렬되었네요 ^^; 아마도 해당부분이 선반영된것 같습니다.

그외, 다듬으면 좋을것 같은 부분들에 대해 코맨트 작성했습니다.
참고 부탁드립니다.

1. 시크릿에 대한 최소한의 접근 권한을 지니도록
[RBAC 규칙을 활성화 또는 구성](/docs/reference/access-authn-authz/authorization/)한다.
1. 특정 컨테이너에서만 시크릿에 접근하도록 한다.
1. [외부 시크릿 저장소 제공 서비스를 사용하는 것을 고려](https://secrets-store-csi-driver.sigs.k8s.io/concepts.html#provider-for-the-secrets-store-csi-driver)한다.
Copy link
Member

Choose a reason for hiding this comment

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

앞선 문장과 이어질수 있도록 고려 보다는 고려한다. 가 조금더 나아보입니다.
참고하셔서 다른 좋은 의견 있으시면 말씀 부탁드립니다.

Copy link
Member Author

@bugoverdose bugoverdose May 25, 2023

Choose a reason for hiding this comment

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

Q. 이 경우 [~고려](~)한다.라는 구조라서 괜찮지 않나요?

Copy link
Member

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.

따라 가 반복되며 문장이 많이 어색합니다.
예를들어,
시크릿을 생성한 방법이나, 파드에서 시크릿을 사용하는 방법에 따라 와 같이 다듬는다면 문서를 읽는데 더 나은 이해를 할수 있을듯합니다.

참고부탁드리며, 좋은 의견 부탁드립니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

시크릿을 어떻게 생성했고 파드에서 시크릿이 어떻게 사용되는지에 따라,는 어떤가요?

Copy link
Member Author

@bugoverdose bugoverdose May 25, 2023

Choose a reason for hiding this comment

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

"방법", "어떻게", "따라"이 두 번 이상 반복안되도록 수정해보았습니다.

e69069b#diff-c1609527b766d7a47d05fbdca289b6e393ee70ec013a6ad3d833091159ac6d63R144

@@ -369,7 +355,7 @@ cat /etc/foo/password
컨테이너의 프로그램은 필요에 따라
이러한 파일에서 시크릿 데이터를 읽는다.

#### 마운트된 시크릿은 자동으로 업데이트됨
#### 탑재된 시크릿은 자동으로 업데이트
Copy link
Member

Choose a reason for hiding this comment

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

문장이 많이 어색합니다.
개인적으로는 마운트된 시크릿의 자동 업데이트 와 같이 해당 주제에 대해 직관적으로 이해가 가능한 문장이 되었으면 좋겟씁니다.

더 좋은 의견이 있으시다면 말씀 부탁드립니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

변경할 수 없다. 컨테이너화된 애플리케이션이 있고
변경하려는 경우, 변경 사항이 포함된 새 이미지를 빌드한
다음, 업데이트된 이미지에서 시작하도록 컨테이너를 다시 생성해야 한다.
컨테이너는 상태를 지니지 않고(stateless)
Copy link
Member

Choose a reason for hiding this comment

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

stateless의 경우 용어집에 발음을 그대로 사용하는 것으로 되어있습니다.
이에 맞추어 수정 부탁드립니다.

https://kubernetes.io/ko/docs/contribute/localization_ko/
stateless , 스테이트리스

Copy link
Member Author

Choose a reason for hiding this comment

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


## 컨테이너 런타임

{{< glossary_definition term_id="container-runtime" length="all" >}}

## {{% heading "whatsnext" %}}
일반적으로 클러스터에서 파드에 대해 기본 컨테이너 런타임을 선택하도록 해도 된다.
Copy link
Member

Choose a reason for hiding this comment

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

문장을 조금 매끄럽게 다듬었으면 좋겠습니다.
개인적으로는 일반적으로 클러스터가 파드의 기본 컨테이너 런타임을 선택하도록 허용가능하다

더 나은 의견이 있으시면 말씀 부탁드립니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

어떻게 해도 어색하네요. 이런 방식은 어떨까요?
e69069b#diff-00a779148ab4f342c926dbec331ea27eb82a66214554742f5390e4163ded18e6R46

Copy link
Member

Choose a reason for hiding this comment

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

원문에서 pickallow 부분이 있어 수정버전의 문장의 뉘앙스가 다르게 느껴집니다.

현재버전 : 일반적으로 클러스터에서 파드의 기본 컨테이너 런타임을 사용하도록 설정해도 문제는 없다.
제안 : 일반적으로 클러스터에서 파드의 기본 컨테이너 런타임을 선택하도록 허용할 수 있다.

의견 부탁드립니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -180,6 +197,18 @@ kubelet이 컨테이너 런타임을 사용하여 파드의 컨테이너 생성
[프라이빗 레지스트리에서 이미지 가져오기](/ko/docs/tasks/configure-pod-container/pull-image-private-registry/)를 참조한다.
해당 예시는 도커 허브에서 제공하는 프라이빗 레지스트리를 사용한다.

### 인증된 이미지 풀을 위한 Kubelet 자격증명 제공자(Credential Provider) {#kubelet-credential-provider}
Copy link
Member

Choose a reason for hiding this comment

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

앞선 문장과 이어질수 있도록 수정되었으면 좋겠습니다.
인증된 이미지를 가져오기 위한

참고 부탁드립니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

{{< /note >}}

kubelet에서 플러그인 바이너리를 호출하여 컨테이너 이미지에 대한 레지스트리 자격증명을 동적으로 가져오도록 설정할 수 있다.
이는 프라이빗 레지스트리에서 자격증명을 가져오는 방법 중 가장 강력하며 휘발성이 있는 방식이지만, 활성화하기 위해 kubelet 수준의 설정이 필요하다.
Copy link
Member

Choose a reason for hiding this comment

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

설정 보다는 구성 이 조금더 이해하는데 도움이 될것 같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

kubelet에서 플러그인 바이너리를 호출하여 컨테이너 이미지에 대한 레지스트리 자격증명을 동적으로 가져오도록 설정할 수 있다.
이는 프라이빗 레지스트리에서 자격증명을 가져오는 방법 중 가장 강력하며 휘발성이 있는 방식이지만, 활성화하기 위해 kubelet 수준의 설정이 필요하다.

세부 사항은 [kubelet 이미지 자격증명 제공자 설정하기](/docs/tasks/administer-cluster/kubelet-credential-provider/)를 참고하라.
Copy link
Member

Choose a reason for hiding this comment

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

타 문서와의 통일성을 위해 다음과 같이 변경 부탁드립니다.

자세한 내용은 ~~~를 참고한다.

https://kubernetes.io/ko/docs/contribute/localization_ko/#%EB%AA%85%EB%A0%B9%ED%98%95

Copy link
Member Author

Choose a reason for hiding this comment

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

@bugoverdose bugoverdose force-pushed the dev-1.26-ko.1-outdated-18-22 branch from b7d1eb2 to e69069b Compare May 25, 2023 12:19
@bugoverdose
Copy link
Member Author

bugoverdose commented May 25, 2023

리뷰 반영 및 질문과 커멘트 남기고, 커밋 하나로 합쳤습니다.

@bugoverdose bugoverdose force-pushed the dev-1.26-ko.1-outdated-18-22 branch from e69069b to 4135336 Compare May 26, 2023 04:47
@ysyukr
Copy link
Member

ysyukr commented May 26, 2023

@bugoverdose 고생하셨습니다 :)
/lgtm

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

LGTM label has been added.

Git tree hash: 038895b83eb7b5d82904918f0b0aed772dbabd55

@ysyukr
Copy link
Member

ysyukr commented May 28, 2023

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ysyukr

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 May 28, 2023
@k8s-ci-robot k8s-ci-robot merged commit 35c1557 into kubernetes:dev-1.26-ko.1 May 28, 2023
@bugoverdose bugoverdose deleted the dev-1.26-ko.1-outdated-18-22 branch June 15, 2023 14:16
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/ko Issues or PRs related to Korean 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.

3 participants