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.22-ko.1 (M52) #29941

Merged

Conversation

seokho-son
Copy link
Member

Partially fix: #29257

  1. M52. content/en/docs/setup/production-environment/tools/kubeadm/control-plane-flags.md | 140(+L) 20(-)

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Oct 6, 2021
@seokho-son seokho-son changed the title Update outdated files in dev-1.22-ko.1 (M52) [ko] Update outdated files in dev-1.22-ko.1 (M52) Oct 6, 2021
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 6, 2021
@k8s-ci-robot k8s-ci-robot added language/ko Issues or PRs related to Korean language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Oct 6, 2021
@seokho-son
Copy link
Member Author

@jihoon-seo 혹시 리뷰 가능하실까요? ^^

@netlify
Copy link

netlify bot commented Oct 6, 2021

✔️ Deploy Preview for k8s-dev-ko ready!

🔨 Explore the source changes: 704a303

🔍 Inspect the deploy log: https://app.netlify.com/sites/k8s-dev-ko/deploys/615ff5428e7f47000796a1d2

😎 Browse the preview: https://deploy-preview-29941--k8s-dev-ko.netlify.app

@jihoon-seo
Copy link
Member

@seokho-son 넵~~
/assign

Copy link
Member

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@seokho-son 빠르게 리뷰해 보았습니다 😊
PTAL!

Comment on lines 187 to 188
kubelet을 사용자 정의하려면, `KubeletConfiguration`을 동일한 구성 파일 내에서 `---`로 구분된 `ClusterConfiguration`이나 `InitConfiguration` 다음에 추가하면
된다. 그러면 해당 파일이 `kubeadm init`에 전달될 수 있다.
Copy link
Member

Choose a reason for hiding this comment

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

[원문]

To customize the kubelet you can add a KubeletConfiguration next to the ClusterConfiguration or InitConfiguration separated by --- within the same configuration file. This file can then be passed to kubeadm init.

Suggested change
kubelet을 사용자 정의하려면, `KubeletConfiguration`을 동일한 구성 파일 내에서 `---`로 구분된 `ClusterConfiguration`이나 `InitConfiguration` 다음에 추가하면
된다. 그러면 해당 파일이 `kubeadm init`에 전달될 수 있다.
kubelet을 사용자 정의하려면, `KubeletConfiguration`을 동일한 구성 파일 내에서 `---`로 구분된 `ClusterConfiguration`이나 `InitConfiguration` 다음에 추가하면
된다. 그런 다음 `kubeadm init` 실행 시에 해당 파일을 파라미터로 전달한다.

Copy link
Member Author

Choose a reason for hiding this comment

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

저는 제안해주신 그런 다음 kubeadm init 실행 시에 해당 파일을 파라미터로 전달한다. 보다는

기존의 그러면 해당 파일이 kubeadm init에 전달될 수 있다. 가 의미상 더 적절할 것 같다는 생각이 듭니다.. ^^

사용자가 직접 전달하는 행위를 하는 것이 아니라,
설정에 의해서 자동으로 kubeadm init 에 해당 파일(내용)이 전달된다는 뜻인 것 같아요.

Copy link
Member

Choose a reason for hiding this comment

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

@seokho-son 제가 파악하기로는..

https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/control-plane-flags/#customizing-the-kubelet

For more details see Configuring each kubelet in your cluster using kubeadm


https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/kubelet-integration/

You can manage the configuration of your kubelets manually, but kubeadm now provides a KubeletConfiguration API type for managing your kubelet configurations centrally.

[KubeletConfiguration object를 정의하는 YAML 파일 예시]

apiVersion: kubelet.config.k8s.io/v1beta1
kind: KubeletConfiguration
clusterDNS:
- 10.96.0.10

https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/kubelet-integration/#configure-kubelets-using-kubeadm

It is possible to configure the kubelet that kubeadm will start if a custom KubeletConfiguration API object is passed with a configuration file like so kubeadm ... --config some-config-file.yaml.
...
When you call kubeadm init, ...
...
When you call kubeadm join, ...


위의 내용들을 고려하면, 플로우가 다음과 같을 것으로 예상됩니다.

  1. 사용자가,
    1. 기존에 ClusterConfiguration이나 InitConfiguration object 등을 정의한 YAML 파일이 있었다면,
      즉, kubeadm ... --config some-config-file.yaml 에 사용했던 some-config-file.yaml 파일이 있었다면,
      --- 로 구분선을 만든 다음 그 밑에 (또는 위에) KubeletConfiguration object 정의 명세를 추가
    2. 기존에 ClusterConfiguration이나 InitConfiguration object 등을 정의한 YAML 파일이 없었다면,
      즉, kubeadm ... --config some-config-file.yaml 에 사용했던 some-config-file.yaml 파일이 없었다면,
      KubeletConfiguration object를 정의하는 YAML 파일을 새로 작성
  2. 사용자가 kubeadm init 또는 kubeadm join 명령을 실행할 때, --config some-config-file.yaml arg를 추가하여 실행
    (a custom KubeletConfiguration API object is passed with a configuration file like so kubeadm ... --config some-config-file.yaml)
  3. kubeadm이 각 노드에 kubelet을 실행시킬 때, 위에서 사용자가 정의했던 KubeletConfiguration object에 대한 명세가 kubelet (내지는 각 노드) 으로 전달됨

저는 이렇게 파악했는데.. 어떻게 보시나요 😊


그리고 이 PR에서 다루고 있는 영어 원문 페이지에서,

  • Kubeadm allows you to pass a directory with patch files to InitConfiguration and JoinConfiguration on individual nodes.
  • You can pass this file to kubeadm init with --config <YOUR CONFIG YAML>:
  • You can pass this file to kubeadm join with --config <YOUR CONFIG YAML>:
  • To customize the kubelet you can add a KubeletConfiguration next to the ClusterConfiguration or InitConfiguration separated by --- within the same configuration file. This file can then be passed to kubeadm init.
  • To customize kube-proxy you can pass a KubeProxyConfiguration next your ClusterConfiguration or InitConfiguration to kubeadm init separated by ---.

저는 이 문장들이 모두 같은 적용 방식 (사용자가 kubeadm init 또는 kubeadm join 명령을 실행할 때, --config some-config-file.yaml arg를 추가하여 실행) 을 의미하는 것으로 보았는데.. 어떻게 보시나요 😊

Copy link
Member Author

Choose a reason for hiding this comment

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

@jihoon-seo 자세히 파악해주셨군요. 감사합니다..^^

You can pass this file to kubeadm join with --config <YOUR CONFIG YAML> 와 같은 문장은 명확이
사용자가 kubeadm init 또는 kubeadm join 명령을 실행할 때, --config some-config-file.yaml arg를 추가하여 실행) 을 의미 하고 있네요.. ^^

아래 문장은 명확하게 적혀있지 않아서,
제안해주신 형태로 의역을 추가할지 고민이 됩니다.
(한글화팀의 가이드(컨벤션?)상 의역을 가능한 줄이고 있어요..^^ 원문에 가깝게~ 이 가이드에 대한 의견이 다르신 경우 정기회의에서 이슈화 부탁드려요.. ^^)

To customize the kubelet you can add a KubeletConfiguration next to the ClusterConfiguration or InitConfiguration separated by --- within the same configuration file. This file can then be passed to kubeadm init.

  • 원문: This file can then be passed to kubeadm init.
  • 기존: 그러면 해당 파일이 kubeadm init에 전달될 수 있다.
  • 리뷰 제안: 그런 다음 kubeadm init 실행 시에 해당 파일을 파라미터로 전달한다.

그런 다음 kubeadm init에 해당 파일을 전달한다. 정도로 번역하면 어떨까요?
(기존의 번역과 그러면 해당 파일이 kubeadm init에 전달될 수 있다. 능동/수동을 바꾼 형태이긴 하네요)

Copy link
Member

Choose a reason for hiding this comment

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

@seokho-son

그런 다음 kubeadm init에 해당 파일을 전달한다. 정도로 번역하면 어떨까요?

네 좋은 것 같습니다! 😊😎😁😉👌👍

@seokho-son seokho-son force-pushed the outdate-1.22-ko.1-m52 branch from 2a21254 to 704a303 Compare October 8, 2021 07:37
@seokho-son
Copy link
Member Author

@jihoon-seo 리뷰 감사합니다. 리뷰에서 협의된 내용에 따라 PR을 업데이트하였습니다.
협의 사항을 모두 반영하였으며, PR을 빠르게 처리하기 위해서 스쿼싱도 함께 처리하였습니다.

@claudiajkang
Copy link
Contributor

/lgtm

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

LGTM label has been added.

Git tree hash: cb933b734fcc6b98e6479e43813672db53b8165b

@gochist
Copy link
Member

gochist commented Oct 9, 2021

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gochist

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 Oct 9, 2021
@k8s-ci-robot k8s-ci-robot merged commit 14e5fe6 into kubernetes:dev-1.22-ko.1 Oct 9, 2021
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.

5 participants