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 korean contents dev-1.24-ko.1(M84-M89) #35564

Conversation

jinnypark9393
Copy link
Contributor

@jinnypark9393 jinnypark9393 commented Jul 30, 2022

M84 ~ M89의 변경사항들을 업데이트하였습니다.

  • M85의 경우, 업데이트 사항이 영문 오타수정으로 한글에는 변경사항이 없습니다.

  • M87의 경우, 변경사항이 문서 가장 마지막의 공백 라인 수의 변경이었는데 이런 변경사항의 경우에도 반영을 하는 것이 맞을까요(이번 커밋에는 원문과 동일하게 문서 마지막의 공백라인수를 맞춰서 변경하였습니다)?

  • M89의 경우, 한글화 정기회의에서 논의한 바와 같이 reviewer란을 기존의 공백 대신, 관련 라인들을 모두 주석 처리하는 것으로 변경하였습니다(M84~M88은 원문에 리뷰어가 기재되어있지 않은 상황).

  • Related Issue: Outdated Korean contents in the dev-1.24-ko.2 #34903

  • M84. content/en/docs/tasks/configure-pod-container/assign-memory-resource.md | 4(+XS) 4(-)
  • M85. content/en/docs/tasks/configure-pod-container/assign-pods-nodes.md | 1(+XS) 1(-)
  • M86. content/en/docs/tasks/configure-pod-container/configure-persistent-volume-storage.md | 2(+XS) 2(-)
  • M87. content/en/docs/tasks/configure-pod-container/configure-pod-initialization.md | 1(+XS) 0(-)
  • M88. content/en/docs/tasks/configure-pod-container/configure-runasusername.md | 4(+XS) 4(-)
  • M89. content/en/docs/tasks/configure-pod-container/static-pod.md | 6(+XS) 6(-)

/language ko

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

netlify bot commented Jul 30, 2022

Deploy Preview for k8s-dev-ko ready!

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

@jinnypark9393 jinnypark9393 force-pushed the 220730-ko-Update-Outdated-Korean-contents-dev-1.24-ko.1-M84-M89 branch from cebaee4 to c3cebab Compare July 30, 2022 05:42
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 30, 2022
@jihoon-seo
Copy link
Member

@jinnypark9393 기여 감사합니다~~

M87의 경우, 변경사항이 문서 가장 마지막의 공백 라인 수의 변경이었는데 이런 변경사항의 경우에도 반영을 하는 것이 맞을까요(이번 커밋에는 원문과 동일하게 문서 마지막의 공백라인수를 맞춰서 변경하였습니다)?

제 경험상 이러한 경우 보통 영문 파일의 변경 사항은
마지막 (또는 파일 중간의) 공백 라인이 여러개였던 것을 하나로 줄이는 변경인 경우가 많았습니다.

개인적으로 저는 영문 파일의 마지막 공백 라인이 몇 개이든 (0 ~ n 개)
한국어 파일의 마지막에는 공백 라인을 1개 넣어서 커밋하고 있습니다.

그리고 Netlify build가 실패했는데요,

@jinnypark9393
Copy link
Contributor Author

jinnypark9393 commented Jul 30, 2022

@jinnypark9393 기여 감사합니다~~

M87의 경우, 변경사항이 문서 가장 마지막의 공백 라인 수의 변경이었는데 이런 변경사항의 경우에도 반영을 하는 것이 맞을까요(이번 커밋에는 원문과 동일하게 문서 마지막의 공백라인수를 맞춰서 변경하였습니다)?

제 경험상 이러한 경우 보통 영문 파일의 변경 사항은 마지막 (또는 파일 중간의) 공백 라인이 여러개였던 것을 하나로 줄이는 변경인 경우가 많았습니다.

개인적으로 저는 영문 파일의 마지막 공백 라인이 몇 개이든 (0 ~ n 개) 한국어 파일의 마지막에는 공백 라인을 1개 넣어서 커밋하고 있습니다.

그리고 Netlify build가 실패했는데요,

그러면 저도 마지막 공백은 1개로 줄여서 다시 �force push 해보겠습니다!

@jinnypark9393 jinnypark9393 force-pushed the 220730-ko-Update-Outdated-Korean-contents-dev-1.24-ko.1-M84-M89 branch from c3cebab to f270b88 Compare July 30, 2022 08:02
@@ -120,7 +120,7 @@ ContainerAdministrator
## {{% heading "whatsnext" %}}


* [쿠버네티스에서 윈도우 컨테이너 스케줄링을 위한 가이드](/ko/docs/setup/production-environment/windows/user-guide-windows-containers/)
* [그룹 매니지드 서비스 어카운트를 이용하여 워크로드 신원 관리하기](/ko/docs/setup/production-environment/windows/user-guide-windows-containers/#그룹-매니지드-서비스-어카운트를-이용하여-워크로드-신원-관리하기)
* [쿠버네티스에서 윈도우 컨테이너 스케줄링을 위한 가이드](/docs/concepts/windows/user-guide/)
Copy link
Member

Choose a reason for hiding this comment

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

#35209 에서 rename 되었기 때문에 링크를 한글 문서로 걸어주셔도 좋을 것 같아요!!

Suggested change
* [쿠버네티스에서 윈도우 컨테이너 스케줄링을 위한 가이드](/docs/concepts/windows/user-guide/)
* [쿠버네티스에서 윈도우 컨테이너 스케줄링을 위한 가이드](/ko/docs/concepts/windows/user-guide/)

* [쿠버네티스에서 윈도우 컨테이너 스케줄링을 위한 가이드](/ko/docs/setup/production-environment/windows/user-guide-windows-containers/)
* [그룹 매니지드 서비스 어카운트를 이용하여 워크로드 신원 관리하기](/ko/docs/setup/production-environment/windows/user-guide-windows-containers/#그룹-매니지드-서비스-어카운트를-이용하여-워크로드-신원-관리하기)
* [쿠버네티스에서 윈도우 컨테이너 스케줄링을 위한 가이드](/docs/concepts/windows/user-guide/)
* [그룹 매니지드 서비스 어카운트를 이용하여 워크로드 신원 관리하기](/docs/concepts/windows/user-guide/#managing-workload-identity-with-group-managed-service-accounts)
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/windows/user-guide/#managing-workload-identity-with-group-managed-service-accounts)
* [그룹 매니지드 서비스 어카운트를 이용하여 워크로드 신원 관리하기](/ko/docs/concepts/windows/user-guide//#그룹-매니지드-서비스-어카운트를-이용하여-워크로드-신원-관리하기)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

알려주셔서 감사합니다! 문서 링크 변경해서 적용하도록 하겠습니다 :)

@seokho-son
Copy link
Member

/assign

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.

@jinnypark9393
한글화 작업 브랜치 관리 기여에 감사드립니다!
역시~ 깔끔하게 잘 처리해주셨네요!

승인에 앞서, 사소하지만, 누락된 사항이 하나 보여서 의견 드립니다.

@@ -291,7 +291,7 @@ Events:
메모리를 표시할 수 있다. E, P, T, G, M, K, Ei, Pi, Ti, Gi, Mi, Ki.
예를 들어 다음은 거의 유사한 값을 나타낸다.

```shell
```
128974848, 129e6, 129M , 123Mi
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
128974848, 129e6, 129M , 123Mi
128974848, 129e6, 129M, 123Mi

요기 마이너하지만 누락된 사항이 있어요 ^^

Copy link
Contributor Author

Choose a reason for hiding this comment

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

리뷰 감사합니다! 말씀주신 수정사항 반영하고 커밋내역 squash하여 commit내역 1개로 정리했습니다 :)

@jinnypark9393 jinnypark9393 force-pushed the 220730-ko-Update-Outdated-Korean-contents-dev-1.24-ko.1-M84-M89 branch from 65e14f1 to b1c9693 Compare August 4, 2022 20:32
@seokho-son
Copy link
Member

Thanks you all ! :)

/lgtm

@seokho-son
Copy link
Member

/approve

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

LGTM label has been added.

Git tree hash: 6bd6482332cd9b7ddb1ed596500add15edc6d97b

@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 Aug 5, 2022
@k8s-ci-robot k8s-ci-robot merged commit 2c7f305 into kubernetes:dev-1.24-ko.2 Aug 5, 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/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants