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 [M23] #41253

Merged

Conversation

bugoverdose
Copy link
Member

from #38458

I have updated one of the outdated files in dev-1.26-ko.1 branch.

Full changes in the original english content can be checked with this comparison.

Updated File

  • M23. content/en/docs/concepts/extend-kubernetes/_index.md | 174(+L) 119(-)

NOTE

It's one page, but it contains a lot of change in content, so it might be take some time to review.

@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 21, 2023
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label May 21, 2023
@netlify
Copy link

netlify bot commented May 21, 2023

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 31795a9
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/646e26fe26f3bd0007b87961
😎 Deploy Preview https://deploy-preview-41253--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.

@seokho-son
Copy link
Member

/assign @seokho-son

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.

@bugoverdose
장문의 문서 업데이트에 고생 많으셨습니다. 감사합니다! :)

앞쪽에 업데이트 누락 사항이 다소 보입니다.
영어 원문의 라인과 한글 문서의 라인 번호를 비교하여, (가급적) 동일하게 맞추며
한번 전반적으로 다시 살펴봐주시면 좋을 것 같습니다.

아래 리뷰 의견 참고하여 업데이트 부탁드립니다.

사용자 정의 방식은 크게 플래그, 로컬 구성 파일 또는 API 리소스 변경만
포함하는 *구성* 과 추가 프로그램이나 서비스 실행과 관련된 *익스텐션* 으로
포함하는 [구성](#구성)과 추가 프로그램이나 서비스 실행을 요구하는 [익스텐션](#익스텐션)으로
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
포함하는 [구성](#구성)추가 프로그램이나 서비스 실행을 요구하는 [익스텐션](#익스텐션)으로
포함하는 [구성](#구성)추가적인 프로그램, 추가적인 네트워크 서비스, 또는 둘 다의 실행을 요구하는 [익스텐션](#익스텐션)으로

변경 사항 일부가 누락된 것 같습니다. :)

dev-1.25-ko.1...dev-1.26-ko.1#diff-a2d97c6562dbf823c05a31e77281c7b79a9844cb07c28845e7ede3274d72ac1eR33

31번: Customization approaches can be broadly divided into [configuration](#configuration), which only
involves changing command line arguments, local configuration files, or API resources; and [extensions](#extensions),
which involve running additional programs, additional network services, or both.
34번: This document is primarily about _extensions_.

문서의 전체 라인을 원문과 동일하게 맞추기 위해서, 아래 33번째 줄에 있는 라인 개행 및,

익스텐션을 _익스텐션_ 으로 변경이 필요합니다.

Copy link
Member Author

@bugoverdose bugoverdose May 23, 2023

Choose a reason for hiding this comment

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

반영하고 줄바꿈 추가했습니다. _익스텐션_이 아니라 *익스텐션*으로 했는데 문제 있으면 수정하겠습니다. (참고)

9365d67#diff-4c545ef3b86efe513900503edb87aea684cc2e130eef306838687973ba50addbR32

## 구성

*구성 파일* 및 *플래그* 는 온라인 문서의 레퍼런스 섹션에
*구성 파일* 및 *명령행 인자* 는 온라인 문서의 [레퍼런스](/docs/reference/) 섹션에
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/reference/) 섹션에
*구성 파일**명령행 인자* 는 온라인 문서의 [레퍼런스](/ko/docs/reference/) 섹션에

한글 문서가 있는 경우, 링크 주소를 대체해주시면 됩니다. :)

Copy link
Member Author

Choose a reason for hiding this comment

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

플래그 및 구성 파일을 항상 변경할 수 있는 것은 아니다. 변경
가능한 경우 일반적으로 클러스터 관리자만 변경할 수 있다. 또한 향후
명령행 인자 및 구성 파일을 항상 변경할 수 있는 것은 아니다. 변경
가능하더라도 일반적으로 클러스터 오퍼레이터를 통해서만 변경될 수 있다. 또한 향후
쿠버네티스 버전에서 변경될 수 있으며, 이를 설정하려면 프로세스를 다시
시작해야 할 수도 있다. 이러한 이유로 다른 옵션이 없는 경우에만 사용해야 한다.

[리소스쿼터](/ko/docs/concepts/policy/resource-quotas/),
[파드시큐리티폴리시(PodSecurityPolicy)](/ko/docs/concepts/security/pod-security-policy/),
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
[파드시큐리티폴리시(PodSecurityPolicy)](/ko/docs/concepts/security/pod-security-policy/),

[PodSecurityPolicies](/docs/concepts/security/pod-security-policy/),

누락된 변경 사항입니다. 삭제가 필요합니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

빌트인 *정책 API(Policy API)* 는 선언적으로 구성된 정책 설정을 제공하는 내장 쿠버네티스 API이다. API는
일반적으로 호스팅된 쿠버네티스 서비스 및 매니지드 쿠버네티스 설치 환경과 함께 사용할 수도 있다.
빌트인 정책 API는 파드와 같은 다른 쿠버네티스 리소스와 동일한 규칙을 따른다.
안정적인 정책 API를 사용하는 경우 다른 쿠버네티스 API와 같이 [정의된 지원 정책](/docs/reference/using-api/deprecation-policy/)의 혜택을 볼 수 있다.
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
안정적인 정책 API를 사용하는 경우 다른 쿠버네티스 API와 같이 [정의된 지원 정책](/docs/reference/using-api/deprecation-policy/)의 혜택을 볼 수 있다.
[안정적인](/ko/docs/reference/using-api/#api-versioning) 정책 API를 사용하는 경우 다른 쿠버네티스 API와 같이 [정의된 지원 정책](/docs/reference/using-api/deprecation-policy/)의 혜택을 볼 수 있다.

변경사항이 누락되어 있습니다.

([RBAC](/docs/reference/access-authn-authz/rbac/)), are built-in Kubernetes APIs that provide declaratively configured policy settings.

Copy link
Member Author

Choose a reason for hiding this comment

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

이를 *웹훅(Webhook)* 이라고 한다. 원격 서비스를 *웹훅 백엔드* 라고 한다. 컨트롤러와
마찬가지로 웹훅은 장애 지점을 추가한다.
컨트롤러는 쿠버네티스 API의 클라이언트이다. 쿠버네티스가 클라이언트이고 원격 서비스를 호출할 때,
쿠버네티스는 이를 *웹훅(Webhook)* 이라고 한다. 원격 서비스를 *웹훅 백엔드* 라고 한다.
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
쿠버네티스는 이를 *웹훅(Webhook)* 이라고 한다. 원격 서비스를 *웹훅 백엔드* 라고 한다.
쿠버네티스는 이를 *웹훅(Webhook)* 이라고 한다. 원격 서비스는 *웹훅 백엔드* 라고 한다.

(그리고) 를 생각한 것이라고 생각하여~ 조금 읽기 편하게 수정해보았습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

통해 새로운 노드 리소스(CPU 및 메모리와 같은 빌트인 자원 외에)를
발견할 수 있게 해준다.
또한 쿠버네티스는 (CSI를 권장하며) v1.23부터 사용 중단된
[FlexVolume](/docs/concepts/storage/volumes/#flexvolume-deprecated) 플러그인에 대한 지원도 포함한다.
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
[FlexVolume](/docs/concepts/storage/volumes/#flexvolume-deprecated) 플러그인에 대한 지원도 포함한다.
[FlexVolume](/ko/docs/concepts/storage/volumes/#flexvolume-deprecated) 플러그인에 대한 지원도 포함한다.

Copy link
Member Author

Choose a reason for hiding this comment

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

파드에 대해 선택된 노드를 필터링하고 우선 순위를 지정할 수 있도록 하는
[웹훅](https://git.k8s.io/design-proposals-archive/scheduling/scheduler_extender.md)을
지원한다.
어떤 [스케줄링 플러그인](/docs/reference/scheduling/config/#scheduling-plugins)을 활성화시킬지 제어하거나,
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/reference/scheduling/config/#scheduling-plugins)을 활성화시킬지 제어하거나,
어떤 [스케줄링 플러그인](/ko/docs/reference/scheduling/config/#scheduling-plugins)을 활성화시킬지 제어하거나,

Copy link
Member Author

Choose a reason for hiding this comment

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

[웹훅](https://git.k8s.io/design-proposals-archive/scheduling/scheduler_extender.md)을
지원한다.
어떤 [스케줄링 플러그인](/docs/reference/scheduling/config/#scheduling-plugins)을 활성화시킬지 제어하거나,
플러그인들을 다른 이름의 [스케줄러 프로파일](/docs/reference/scheduling/config/#multiple-profiles)과 연결지을 수도 있다.
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/reference/scheduling/config/#multiple-profiles)과 연결지을 수도 있다.
플러그인들을 다른 이름의 [스케줄러 프로파일](/ko/docs/reference/scheduling/config/#여러-프로파일)과 연결지을 수도 있다.

Copy link
Member Author

Choose a reason for hiding this comment

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

Comment on lines 299 to 300
다른 익스텐션 포인트는
웹훅 통합으로 제공되지 않는다.
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
다른 익스텐션 포인트는
웹훅 통합으로 제공되지 않는다.
다른 익스텐션 포인트는 웹훅 통합으로 제공되지 않는다.

원문과 라인을 (가급적) 동일하게 만들기 위한 제안입니다.
(문서 전반적으로 확인이 필요합니다.)

You can only affect node filtering
and node prioritization with a scheduler extender webhook; other extension points are
not available through the webhook integration.

Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -182,7 +182,7 @@ profiles:
볼륨 제한이 충족될 수 있는지 확인한다.
익스텐션 포인트: `filter`.

### 여러 프로파일
### 여러 프로파일 {#multiple-profiles}
Copy link
Member

Choose a reason for hiding this comment

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

한글 앵커 명칭을 사용하면 되므로, 특별히 추가로 영문 앵커를 붙일 필요는 없을 것 같습니다. :)

적용하시는 경우, 아래 수정사항들은 다시 원복해주시면 되겠습니다.
content/ko/docs/concepts/scheduling-eviction/assign-pod-node.md
content/ko/docs/reference/scheduling/config.md

Copy link
Member Author

Choose a reason for hiding this comment

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

Q. 어디서는 별도로 영문 앵커를 사용하고, 어디서는 한글 앵커를 사용하는 것 같았는데, 특별한 기준이 있는건가요? (ex. 원문쪽에서 앵커를 지정하는 경우 한글쪽도 영문 앵커 활용) 아니면 그냥 여러명이 작업하는 과정에서 자연스럽게 그때그때 작업하는 사람이 판단한 결과인가요?

Copy link
Member

Choose a reason for hiding this comment

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

@bugoverdose

기존 원문에 이미 특별히 영문 앵커가 지정되어 있는 경우가 있습니다. (URL에 원하는 형태로 표시되도록)
이 경우, 해당 앵커는 한글 문서에서도 그대로 영문으로 준용하고 있습니다.

그 이외에 일반적인 경우에는
헤더 명칭이 한글로 변경되고, 한글 헤더에 따라 자동으로 앵커의 주소가 생성되므로,
한글로 자동 생성된 앵커의 주소를 활용하여, 다른 문서 내에서 참조로 활용하고 있습니다.

다만, 꼼꼼하게 확인하지 못한 경우에는 참조에 기존 영문 앵커가 표시된 경우도 있을 것 같습니다. (어쨌든 참조 링크에 오류가 있다면 무조건 수정이 필요합니다.)

해당 사항은 한글화팀에서 협의하여 결정한 사항이나, 절대적인 것은 아니고 필요에 따라 개선해나갈 수 있는 부분 입니다. :)

Copy link
Member Author

Choose a reason for hiding this comment

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

확인했습니다. 일단 영어 앵커 제거하는 방향으로 롤백했습니다.
9365d67#diff-56f24224d555fcb4a18dc3f8a3156ee0fc6e6b5e2a381a74bd564f91d8b2313eR185

Copy link
Member Author

Choose a reason for hiding this comment

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

어쩌다보니 다른 파일에서 잘못 사용되던 영문 앵커도 한글로 수정했습니다.
31795a9#diff-a7a526f741d643d4edc8faa9e0d3e2c471535ec329d73c00272667b69888fc09R217

@bugoverdose
Copy link
Member Author

bugoverdose commented May 23, 2023

원문 323줄과 문단 단위로 줄바꿈을 완벽하게 일치시켜봤는데, 조금 부자연스러운 위치에 줄바꿈을 추가하게 되어서 오히려 이게 적절한 방법인지 살짝 의문도 드네요.
dfc39e5

@seokho-son
Copy link
Member

원문 323줄과 문단 단위로 줄바꿈을 완벽하게 일치시켜봤는데, 조금 부자연스러운 위치에 줄바꿈을 추가하게 되어서 오히려 이게 적절한 방법인지 살짝 의문도 드네요. dfc39e5

@bugoverdose
감사합니다! :)
원문과 완전히 동일하게 일치시킬 필요는 없으나,
문단 별로 구분이 쉽도록 각 줄을 유사하게 만들어 주시는 것은 리뷰와 갱신 필요 항목을 추적 시 큰 도움이 됩니다.

  • 리뷰어는 PR 리뷰시, 제안된 한글 문서 내용만 검토하는 것이 아니라 원문을 함께 살펴봅니다. 원문과 한글 문서의 각 줄을 비교하면, 한글 문서에 해당하는 원문 파악이 쉽고, 누락된 부분을 빠르게 검토할 수 있습니다.
  • 원문이 변경되는 경우 git diff 시 줄 단위로 변경 사항이 표시되므로, 한글 문서에서의 변경 필요 사항을 좀 더 빨리 파악 및 적용할 수 있습니다.

단, 이를 모든 PR에 대해 의무화하고 있지는 않고, 권장 사항으로 처리하고 있습니다.
본 PR은 기존 한글 문서를 유지보수하기 위한 갱신 PR이라, 가급적 적용해주시는 것이 좋습니다. (갱신 누락 사항 파악겸)

관련하여, 혹시 더 좋은 방안 및 아이디어가 있으시면 공유 부탁드립니다! :)

@seokho-son
Copy link
Member

@bugoverdose
빠르고 꼼꼼한 업데이트! 감사합니다. :)
다양한 의견을 주시는 것을 보니, 앞으로의 기여도 기대가 됩니다. ㅎㅎ

이탤릭 마크다운 관련 의견 이외의 사항은 모두 확인 완료하였습니다. (lgtm)
승인을 위해, 추가 작업을 포함하여 커밋을 1개로 만들어주시면 되겠습니다.

@bugoverdose bugoverdose force-pushed the dev-1.26-ko.1-outdated-23 branch from f8b13f0 to 31795a9 Compare May 24, 2023 15:02
@bugoverdose
Copy link
Member Author

리뷰 내용 전부 반영하고 커밋 하나로 합쳤습니다.

@seokho-son
Copy link
Member

/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 25, 2023
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: ab7402450f459bc9679101d1253ea82e67972c55

@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 May 25, 2023
@k8s-ci-robot k8s-ci-robot merged commit cb71d71 into kubernetes:dev-1.26-ko.1 May 25, 2023
@bugoverdose bugoverdose deleted the dev-1.26-ko.1-outdated-23 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