Skip to content

Conversation

@yujinchoi-94
Copy link
Contributor

  • M59. content/en/docs/concepts/services-networking/dns-pod-service.md | 79(+M) 61(-)

@k8s-ci-robot
Copy link
Contributor

Welcome @yujinchoi-94!

It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/website has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 27, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: yujinchoi-94 / name: YujinChoi (6ea2038)

@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. language/ko Issues or PRs related to Korean language labels Feb 27, 2023
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Feb 27, 2023
@yujinchoi-94 yujinchoi-94 changed the base branch from main to dev-1.26-ko.1 February 27, 2023 14:16
@k8s-ci-robot k8s-ci-robot added 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. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Feb 27, 2023
@netlify
Copy link

netlify bot commented Feb 27, 2023

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 6ea2038
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/63fcbb229376e00008ea0bff
😎 Deploy Preview https://deploy-preview-39700--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 settings.

Comment on lines 54 to 55
요약하면, _test_ 네임스페이스에 있는 파드는 `data.prod` 또는
`data.prod.svc.cluster.local` 중 하나를 통해 성공적으로 해석될 수 있다.
`data.prod.svc.cluster.local` 성공적으로 해석할 수 있다.
Copy link
Member

@jihoon-seo jihoon-seo Feb 28, 2023

Choose a reason for hiding this comment

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

@yujinchoi-94 기여 감사합니다!

@bconfiden2 님의 의견 도 참고하여 다음과 같이 제안해 봅니다 ㅎㅎ

요약하면, _test_ 네임스페이스에 있는 파드는, `prod` 네임스페이스에 있는 `data` 파드를 가리키는 도메인 이름인 
`data.prod` 또는 `data.prod.svc.cluster.local` 을 성공적으로 해석할 수 있다.

Copy link
Contributor Author

@yujinchoi-94 yujinchoi-94 Feb 28, 2023

Choose a reason for hiding this comment

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

리뷰 감사합니다. 의견 주신대로 수정했습니다 😀

Copy link
Contributor

@jmyung jmyung left a comment

Choose a reason for hiding this comment

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

빠르게 검토했습니다. 수고 많으셨습니다. 👍


"헤드리스"(클러스터 IP가 없는) 서비스 또한 서비스 IP 계열에 따라
`my-svc.my-namespace.svc.cluster-domain.example`
[헤드리스 서비스](/docs/concepts/services-networking/service/#headless-services)(클러스터 IP가 없는) 또한`my-svc.my-namespace.svc.cluster-domain.example`
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
[헤드리스 서비스](/docs/concepts/services-networking/service/#headless-services)(클러스터 IP가 없는) 또한`my-svc.my-namespace.svc.cluster-domain.example`
[헤드리스 서비스](/docs/concepts/services-networking/service/#headless-services)(클러스터 IP가 없는) 또한 `my-svc.my-namespace.svc.cluster-domain.example`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

꼼꼼한 확인 감사합니다 :)

@yujinchoi-94 yujinchoi-94 force-pushed the 220227_update_outdated_M59 branch from 8adf48d to b2b2e43 Compare March 2, 2023 11:40
@seokho-son
Copy link
Member

아래와 같은 이유로, 빌드오류가 발생하고 있습니다. (cve-feed 관련 오류)
해당 PR의 수정사항에 의한 오류는 아니니 참고하시기 바랍니다.
https://kubernetes.slack.com/archives/CA1MMR86S/p1677735915886899

해당 오류 해결을 위해 PR #39787을 통해서 dev-1.26-ko.1가 업데이트될 예정이며,
PR #39787 이 dev-1.26-ko.1에 반영(1~2일 소요 예상)되고 나면,
본 PR의 피쳐 브랜치도 최신 dev-1.26-ko.1을 기준으로 rebase(업데이트)가 필요합니다. 향후에 요청드리도록 하겠습니다.

@kubernetes/sig-docs-ko-reviews 현재 PR 상태에서도 리뷰가 가능하니, self-assign 하여 리뷰 추진을 부탁드립니다. :)

@yujinchoi-94 yujinchoi-94 force-pushed the 220227_update_outdated_M59 branch from b2b2e43 to 1b9d33b Compare March 12, 2023 14:00
@netlify
Copy link

netlify bot commented Mar 12, 2023

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 2b8112d
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/64167fdb42bb42000847e29f
😎 Deploy Preview https://deploy-preview-39700--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
Member

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

안녕하세요 @yujinchoi-94 님!

기여 감사합니다! 의미를 더 명확하게 잘 정리해주셨네요.
한글화 방침과 관련된 몇 가지 의견이 있어서 의견을 남겼습니다.

이미 두 리뷰어께서 주신 의견은 반영을 해주신 것 같으니, 제가 드리는 의견을 최종 의견으로 보시면 될 것 같습니다. 적절히 반영해주시면, 제가 살펴보고 승인까지 처리하도록 하겠습니다.

/assign @seokho-son


"헤드리스"(클러스터 IP가 없는) 서비스 또한 서비스 IP 계열에 따라
`my-svc.my-namespace.svc.cluster-domain.example`
[헤드리스 서비스](/docs/concepts/services-networking/service/#headless-services)(클러스터 IP가 없는) 또한 `my-svc.my-namespace.svc.cluster-domain.example`
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
[헤드리스 서비스](/docs/concepts/services-networking/service/#headless-services)(클러스터 IP가 없는) 또한 `my-svc.my-namespace.svc.cluster-domain.example`
[헤드리스 서비스](/ko/docs/concepts/services-networking/service/#헤드리스-headless-서비스)(클러스터 IP가 없는) 또한 `my-svc.my-namespace.svc.cluster-domain.example`

한글 문서가 있는 경우, 해당 링크를 사용하도록 처리하고 있습니다. :)


엔드포인트 오브젝트는 `hostname` 필드를
임의의 엔드포인트 IP 주소로 지정할 수 있다.
{{<glossary_tooltip term_id="endpoint-slice" text="EndpointSlice">}}는 임의의 엔드포인트 주소에 대해 해당 IP와 함께 DNS 호스트 네임을 지정할 수 있다.
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
{{<glossary_tooltip term_id="endpoint-slice" text="EndpointSlice">}}는 임의의 엔드포인트 주소에 대해 해당 IP와 함께 DNS 호스트 네임을 지정할 수 있다.
{{<glossary_tooltip term_id="endpoint-slice" text="엔드포인트슬라이스(EndpointSlice)">}}는 임의의 엔드포인트 주소에 대해 해당 IP와 함께 DNS 호스트 네임을 지정할 수 있다.

쿠버네티스 문서 한글화 가이드 - 한글화 용어집 에 따라 수정해보았습니다.

EndpointSlice 엔드포인트슬라이스(EndpointSlice) API 오브젝트인 경우

참고로 glossary_tooltip (shortCode)은 https://kubernetes.io/ko/docs/reference/glossary/?all=true 에서 term_id에 해당하는 용어를 매핑하여 출력하며, 해당 용어가 한글화되어 있는 경우 한글화된 용어로 출력하게 되어 있습니다.

단, glossary_tooltip 에 text=가 지정되어 있는 경우 text에 해당하는 값을 출력하므로, 이 경우 text 값은 한글로 변경하고 있습니다.


쿠버네티스는 파드의 DNS 환경 설정을 위해 기본적으로 최대 6개의 탐색 도메인과
최대 256자의 탐색 도메인 목록을 허용한다.
쿠버네티스는 탐색 도메인 리스트가 32개를 넘거나 모든 탐색 도메인의 길이가 2048자를 초과할 때 까지 DNS Config에 자체적인 제한을 하지 않는다.
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
쿠버네티스는 탐색 도메인 리스트가 32개를 넘거나 모든 탐색 도메인의 길이가 2048자를 초과할 때 까지 DNS Config에 자체적인 제한을 하지 않는다.
쿠버네티스는 탐색 도메인 리스트가 32개를 넘거나 모든 탐색 도메인의 길이가 2048자를 초과할 때까지 DNS Config에 자체적인 제한을 하지 않는다.

쿠버네티스는 최대 32개의 탐색 도메인과
최대 2048자의 탐색 도메인 목록을 허용한다.
{{< note >}}
이전 버전의 일부 컨테이너 런타임은 DNS 탐색 도메인 수에 대해 자체적인 제한을 가지고 있을 수 있다. 컨테이너 런타임 환경에 따라 많은 수의 DNS 검색 도메인을 갖는 파드는 pending 상태로 고착될 수 있다.
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
이전 버전의 일부 컨테이너 런타임은 DNS 탐색 도메인 수에 대해 자체적인 제한을 가지고 있을 수 있다. 컨테이너 런타임 환경에 따라 많은 수의 DNS 검색 도메인을 갖는 파드는 pending 상태로 고착될 수 있다.
이전 버전의 일부 컨테이너 런타임은 DNS 탐색 도메인 수에 대해
자체적인 제한을 가지고 있을 수 있다. 컨테이너 런타임
환경에 따라 많은 수의 DNS 검색 도메인을 갖는 파드는
pending 상태로 고착될 수 있다.

https://kubernetes.io/ko/docs/contribute/localization_ko/ 한글화 가이드에 따라

문서 코딩 가이드
가로폭은 원문을 따름
유지보수의 편의를 위해서 원문의 가로폭을 따른다.

즉, 원문이 한 문단을 줄바꿈하지 않고 한 행에 길게 기술했다면 한글화 시에도 한 행에 길게 기술하고, 원문이 한 문단을 줄바꿈해서 여러 행으로 기술한 경우에는 한글화 시에도 가로폭을 원문과 비슷하게 유지한다.

개행이 필요해 보입니다. 이는 코드의 유지보수를 위한 요청사항입니다.
예를 들어, 해당 내용의 영어 원문이 336 ~ 339 라인에 위치해 있다면, 한글 내용도 336 ~ 339 라인에 위치하도록 변경이 필요합니다. (의미에 맞춰 개행할 필요까지는 없고, 대략 라인을 맞추면 됩니다.)

336: Some container runtimes of earlier versions may have their own restrictions on
337: the number of DNS search domains. Depending on the container runtime
338: environment, the pods with a large number of DNS search domains may get stuck in
339: the pending state.

가능하시다면, 문서 전반적으로 영어 원문과 라인이 유사(가급적 동일)하게 되도록
정리해주실 수 있을까요? :)

@yujinchoi-94 yujinchoi-94 force-pushed the 220227_update_outdated_M59 branch from 47c1787 to 2b8112d Compare March 19, 2023 03:22
@yujinchoi-94
Copy link
Contributor Author

@seokho-son 말씀해주신 부분 + 문서 전반적인 라인 수 맞추기 반영했습니다. 리뷰 감사합니다! 😀

@seokho-son
Copy link
Member

@yujinchoi-94 빠른 업데이트 감사합니다! 완벽하네요 :)
리뷰가 길어졌는데 기다려주셔서 감사합니다. PR이 많이 밀려있어서, 하나씩 처리해나가고 있습니다. :)

/lgtm

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

LGTM label has been added.

Git tree hash: 7f36b6a75098845ff458298f3cca19574a9d7c44

@seokho-son
Copy link
Member

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: seokho-son

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 Mar 20, 2023
@k8s-ci-robot k8s-ci-robot merged commit c60480e into kubernetes:dev-1.26-ko.1 Mar 20, 2023
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