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.24-ko.2 (M27-M30) #35677

Merged

Conversation

seokho-son
Copy link
Member

Signed-off-by: Seokho Son shsongist@gmail.com

Resolve tasks (M27-M30) in #34903

@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 Aug 3, 2022
@k8s-ci-robot k8s-ci-robot added the language/ko Issues or PRs related to Korean language label Aug 3, 2022
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Aug 3, 2022
@seokho-son
Copy link
Member Author

@kubernetes/sig-docs-ko-reviews 빠른 리뷰를 부탁 드립니다. ;)

@netlify
Copy link

netlify bot commented Aug 3, 2022

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 7f083cc
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/62eb2ff5549d580008b95e66
😎 Deploy Preview https://deploy-preview-35677--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.

Copy link
Contributor

@yoonian yoonian 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 간단히 살펴보았습니다. 확인 부탁드립니다.

@@ -226,6 +226,7 @@ DNS 정책은 파드별로 설정할 수 있다.
확인할 수 있다.
- "`ClusterFirstWithHostNet`": hostNetwork에서 running 상태인 파드의 경우 DNS 정책인
"`ClusterFirstWithHostNet`"을 명시적으로 설정해야 한다.
- 참고: 윈도위에서는 지원되지 않는다.상세 정보는 [아래](#dns-windows)에서 확인한다.
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
- 참고: 윈도위에서는 지원되지 않는다.상세 정보는 [아래](#dns-windows)에서 확인한다.
- 참고: 윈도우(Windows)에서는 지원하지 않는다. 상세 정보는 [아래](#dns-windows)에서 확인한다.

처음 나온 단어라 의미가 햇갈릴 것 같습니다. 한영 병기해야 할 것 같습니다.

아래 내용을 볼 때, 윈도 보다는 윈도우가 맞을 것 같습니다. 그렇지만, 이 단어는 고유 명사이고, 마이크로소프트 국문 홈페이지에서 Windows 라고 쓰고 있어서, 음차 말고 영어로만 쓰는 것은 어떨까요? 일단은 윈도우로 하고 회의에서 결정하면 좋을 것 같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

@yoonian 의견 감사합니다. :)

Windows (=윈도우)는 한글화팀 회의 및 PR을 통해서, 한글화 가이드에 포함된 용어 입니다.

따라서 오타만 수정하고, 현재 상태로 유지하면 어떨까 싶습니다. (윈도위 -> 윈도우)

용어 가이드 관련해서 혹시 개선이 필요하다고 생각하시면,
PR을 통해 제안 또는 한글화팀 회의 등에서 논의되면 좋을 것 같습니다. :)

@@ -323,6 +324,24 @@ kube-apiserver와 kubelet에 `ExpandedDNSConfig` 기능 게이트가 활성화
쿠버네티스는 최대 32개의 탐색 도메인과
최대 2048자의 탐색 도메인 목록을 허용한다.

## 윈도우 노드에서 DNS 해석(resolution) {#dns-windows}

- ClusterFirstWithHostNet은 윈도우 노드에서 구동 중인 파드에는 지원되지 않는다.
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
- ClusterFirstWithHostNet은 윈도우 노드에서 구동 중인 파드에는 지원되지 않는다.
- ClusterFirstWithHostNet은 윈도우 노드에서 구동 중인 파드에는 지원하지 않는다.

능동표현으로 해보았습니다만, 사실 윈도우가 지원하지 않는 거라 수동형을 놔두는 것이 맞는 것 같지만, 한번 의견 던져 봅니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

주어가 생략되어 있어서 능동으로 표현하면 다소 어색한 것 같기도 하네요. :)

(생략: 쿠버네티스는) 윈도우 노드에서 구동 중인 파드에는 ClusterFirstWithHostNet를 지원하지 않는다.

또는 기존대로 유지하면 좋을 것 같습니다. :)
어떻게 생각하시나요?

Copy link
Contributor

Choose a reason for hiding this comment

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

주어 생략 표현을 자주 보다 보니 이런 코멘트를 남겼나 봅니다. 😅
yuji 하지요.

[`Resolve-DNSName`](https://docs.microsoft.com/powershell/module/dnsclient/resolve-dnsname)
파워쉘(powershell) cmdlet을 사용하는 것을 추천한다.
- 리눅스에는 DNS 접미사 목록이 있는데, 이는 네임이 완전한 주소가 아니어서 주소
해석에 실패한 경우 사용된다.
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
해석에 실패한 경우 사용된다.
해석에 실패한 경우 사용한다.

능동형으로 해도 무리 없을 것 같습니다.

해석에 실패한 경우 사용된다.
윈도우에서는 파드의 네임스페이스(예: `mydns.svc.cluster.local`)와 연계된
하나의 DNS 접미사만 가질 수 있다. 윈도우는 이러한 단일 접미사 통해 해석될 수 있는
FQDNs, 서비스, 또는 네트워크 네임을 해설할 수 있다. 예를 들어, `default`에
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
FQDNs, 서비스, 또는 네트워크 네임을 해설할 수 있다. 예를 들어, `default`
FQDNs, 서비스, 또는 네트워크 네임을 해석할 수 있다. 예를 들어, `default`

해석으로 계속 쓰고 있어서 말입니다.

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:
* `--cloud-provider`가 명시되지 않았다면 관리자는 해당 노드에 듀얼 스택
`.status.addresses`를 수동으로 설정하기 위해 쉼표로 구분된 IP 주소 쌍을 `--node-ip` 플래그로 전달할 수 있다.
해당 노드의 파드가 HostNetwork 모드로 실행된다면,
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
해당 노드의 파드가 HostNetwork 모드로 실행된다면,
해당 노드의 파드를 HostNetwork 모드로 실행한다면,

능동형으로 바꿔도 될 것 같습니다.

1. 서비스를 단일 스택에서 이중 스택으로 변경하려면 원하는 대로 `.spec.ipFamilyPolicy`를
`SingleStack`에서 `PreferDualStack` 또는 `RequireDualStack`으로 변경한다.
이 서비스를 단일 스택에서 이중 스택으로 변경하면 쿠버네티스는 누락된 주소 계열의
것을 배정하므로 해당 서비스는 이제 IPv4와 IPv6 주소를 갖게된다.
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
것을 배정하므로 해당 서비스는 이제 IPv4와 IPv6 주소를 갖게된다.
것을 배정하므로 해당 서비스는 이제 IPv4와 IPv6 주소를 갖는다.

Comment on lines 298 to 299
서비스로 되어 있는 파드와 노드에 대해서는 듀얼 스택(dual-stack) IPv4/IPv6 네트워킹이
지원된다.
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
서비스로 되어 있는 파드와 노드에 대해서는 듀얼 스택(dual-stack) IPv4/IPv6 네트워킹이
지원된다.
서비스로 되어 있는 파드와 노드에 대해서는 듀얼 스택(dual-stack) IPv4/IPv6 네트워킹을
지원한다.

@jihoon-seo
Copy link
Member

/assign @yoonian

Signed-off-by: Seokho Son <shsongist@gmail.com>
@seokho-son
Copy link
Member Author

@yoonian 리뷰 감사합니다!

논의 된 내용으로 업데이트 완료하였습니다.
수정 사항이 많지 않아서, 기존 커밋에 amend 하였습니다. :)

@yoonian
Copy link
Contributor

yoonian commented Aug 4, 2022

/lgtm

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

LGTM label has been added.

Git tree hash: 9dbb1fb219f7efcc33795a345f442b938433383f

@jihoon-seo
Copy link
Member

/assign

@jihoon-seo
Copy link
Member

@seokho-son @yoonian 감사합니다!
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jihoon-seo

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 Aug 6, 2022
@k8s-ci-robot k8s-ci-robot merged commit 9837949 into kubernetes:dev-1.24-ko.2 Aug 6, 2022
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.

None yet

4 participants