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] translate 'cluster-upgrade' page #35441

Merged
merged 3 commits into from
Aug 24, 2022
Merged

[ko] translate 'cluster-upgrade' page #35441

merged 3 commits into from
Aug 24, 2022

Conversation

Kid-Chang
Copy link
Member

@Kid-Chang Kid-Chang commented Jul 27, 2022

#35239

직전 PR에서 이슈가 있어 close하고 새롭게 PR보냅니다.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. language/ko Issues or PRs related to Korean language labels Jul 27, 2022
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Jul 27, 2022
@netlify
Copy link

netlify bot commented Jul 27, 2022

Deploy Preview for k8s-dev-ko ready!

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

@bconfiden2 bconfiden2 left a comment

Choose a reason for hiding this comment

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

부족하지만 개인적인 생각으로 리뷰해보았습니다.. 감사합니다!! 🥺

kubectl convert -f pod.yaml --output-version v1
```

`kubectl` 도구는 `pod.yaml`의 내용을 `kind`를 Pod(변경되지 않음)로 설정하는 매니페스트로 대체하고, 수정된 `apiVersion`으로 대체한다.
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
`kubectl` 도구는 `pod.yaml`의 내용을 `kind`Pod(변경되지 않음)로 설정하는 매니페스트로 대체하고, 수정된 `apiVersion`으로 대체한다.
`kubectl` 도구는 `pod.yaml`의 내용을 `kind`파드(변경되지 않음)로 설정하는 매니페스트로 대체하고,
수정된 `apiVersion`으로 대체한다.

파드를 한글화하였고, 영어 문서와 맞게끔 개행을 하였습니다!

- {{< glossary_tooltip text="컨트롤 플레인" term_id="control-plane" >}} 업그레이드하기
- 클러스터 내부의 노드를 업그레이드하기
- {{< glossary_tooltip text="kubectl" term_id="kubectl" >}}와 같은 클라이언트 업그레이드하기
- 새로운 쿠버네티스 버전에 동반되는 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
- 새로운 쿠버네티스 버전에 동반되는 API 변화에 기반하여 내용과 다른 리소스를 조정하기
- 새로운 쿠버네티스 버전에 동반되는 API 변화에 기반하여, 매니페스트 또는 다른 리소스를 조정하기

뭔가 내용 & 다른 리소스 라는 느낌보다는 "내용과 다른" 리소스 라는 느낌이 들어, 표현을 수정해보았습니다!! 😃


클러스터가 'kubeadm' 도구를 사용하여 배포된 경우
클러스터 업그레이드 방법에 대한 자세한 내용은
[kubeadm 클러스터 업그레이드](/ko/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade/)를 참조하십시오.
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
[kubeadm 클러스터 업그레이드](/ko/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade/)참조하십시오.
[kubeadm 클러스터 업그레이드](/ko/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade/)참조한다.

### 다른 방식의 배포 {#upgrade-other}

권장하는 유지 관리 설정 단계에 대한 자세한 내용은
클러스터 배포 도구 설명서를 참조하십시오.
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
클러스터 배포 도구 설명서를 참조하십시오.
클러스터 배포 도구 설명서를 참조한다.

@Kid-Chang
Copy link
Member Author

@bconfiden2 리뷰 확인했습니다. 명일까지 리뷰주신 내용토대로 수정하겠습니다! 🙏🙏

@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.

@Kid-Chang 님~! 기여 감사합니다!!

전반적으로 살펴보고 리뷰 의견을 드립니다. 의견 참고하셔서 PR 업데이트를 부탁 드립니다. :)
(마일스톤 교체 시기인데 많은 PR들이 열려있어서, 차근차근 진행하느라 리뷰가 지연된 점 양해 부탁드립니다.)

Comment on lines 6 to 8
<!-- overview -->

이 페이지는 쿠버네티스 클러스터를 업그레이드 하기 위해 따라야할 단계에 대한 개요를 제공한다.
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
<!-- overview -->
이 페이지는 쿠버네티스 클러스터를 업그레이드 하기 위해 따라야할 단계에 대한 개요를 제공한다.
<!-- overview -->
이 페이지는 쿠버네티스 클러스터를 업그레이드하기 위해 따라야 할 단계에 대한
개요를 제공한다.
  • 원문에는 없는 라인이어서 삭제를 제안 드립니다.
    (참고로, 영어 원문과 한글에서의 각 라인을 가급적 동일하게 가져가고 있습니다. 코드 유지보수 차원)
  • 띄어쓰기 일부 수정

Comment on lines 10 to 11
클러스터를 업그레이드하는 방법은 처음에 배포한 방법에 의존하며,
그 이후의 변화에도 적용된다.
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
클러스터를 업그레이드하는 방법은 처음에 배포한 방법에 의존하며,
그 이후의 변화에도 적용된다.
클러스터를 업그레이드하는 방법은 초기 배포 방법에 의존적이며,
배포 이후 관련된 변경 사항에도 의존적일 수 있다.

원문을 고려하여 자연스럽게 수정해보았습니다.

The way that you upgrade a cluster depends on how you initially deployed it
and on any subsequent changes.

클러스터를 업그레이드하는 방법은 처음에 배포한 방법에 의존하며,
그 이후의 변화에도 적용된다.

더 높은 수준에서, 수행해야할 단계:
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 text="컨트롤 플레인" term_id="control-plane" >}} 업그레이드하기
- 클러스터 내부의 노드를 업그레이드하기
- {{< glossary_tooltip text="kubectl" term_id="kubectl" >}}와 같은 클라이언트 업그레이드하기
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 text="kubectl" term_id="kubectl" >}} 같은 클라이언트 업그레이드하기
- {{< glossary_tooltip text="kubectl" term_id="kubectl" >}} 같은 클라이언트 업그레이드하기

kubectl 을 쿠브컨트롤 이라고 발음하는 것을 고려하면,
와 보다는 과. 가 좋을 것 같습니다. :)

- {{< glossary_tooltip text="컨트롤 플레인" term_id="control-plane" >}} 업그레이드하기
- 클러스터 내부의 노드를 업그레이드하기
- {{< glossary_tooltip text="kubectl" term_id="kubectl" >}}와 같은 클라이언트 업그레이드하기
- 새로운 쿠버네티스 버전에 동반되는 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
- 새로운 쿠버네티스 버전에 동반되는 API 변화에 기반하여, 매니페스트 또는 다른 리소스를 조정하기
- 새로운 쿠버네티스 버전에 동반되는 API 변화에 기반하여, 매니페스트 또는
다른 리소스를 조정하기

Copy link
Member

Choose a reason for hiding this comment

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

원문과 개행을 맞추기 위함입니다.

이 때 [`kubectl`최신 버전을 설치](/ko/docs/tasks/tools/)
해야한다.

클러스터의 각 노드에 대해 해당 노드를 [드레인](/docs/tasks/administer-cluster/safely-drain-node/)
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/tasks/administer-cluster/safely-drain-node/)
클러스터의 각 노드에 대해 해당 노드를 [드레인(drain)](/docs/tasks/administer-cluster/safely-drain-node/)

영문을 병기하면 좋을 것 같습니다. :)

Comment on lines 63 to 64
권장하는 유지 관리 설정 단계에 대한 자세한 내용은
클러스터 배포 도구 설명서를 참조한다.
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
권장하는 유지 관리 설정 단계에 대한 자세한 내용은
클러스터 배포 도구 설명서를 참조한다.
사용한 클러스터 배포 도구에 따라 해당 배포 도구가
제공하는 문서를 통하여, 유지 관리를 위해 권장되는 설정 단계를 확인한다.

원문과 의미가 살짝 다른 듯하여, 개선해보았습니다. :)

Refer to the documentation for your cluster deployment tool to learn the recommended set
up steps for maintenance.

Comment on lines 70 to 71
클러스터에서 활성화된 쿠버네티스 리소스의
클러스터 내부 표현을 위해 etcd로 직렬화된 객체는
특정 버전의 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
클러스터에서 활성화된 쿠버네티스 리소스의
클러스터 내부 표현을 위해 etcd로 직렬화된 객체는
특정 버전의 API를 사용하여 작성된다.
클러스터에서 활성화된 쿠버네티스 리소스를
클러스터 내부적으로 표현(representation)하기 위해서 etcd로 직렬화된 객체는
특정 버전의 API를 사용하여 작성된다.

원문을 고려하여 개선해보았습니다.

Comment on lines 78 to 79
영향을 받는 각 개체에 대해 지원되는 최신 API를 사용하여
가져온 다음 지원되는 최신 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
영향을 받는 각 개체에 대해 지원되는 최신 API를 사용하여
가져온 다음 지원되는 최신 API를 사용하여 다시 쓴다.
영향을 받는 각 객체를 지원되는 최신 API를 사용하여
가져온(fetch) 다음, 해당 API를 사용하여 다시 쓴다.

kubectl convert -f pod.yaml --output-version v1
```

`kubectl` 도구는 `pod.yaml`의 내용을 `kind`를 파드(변경되지 않음)로 설정하는 매니페스트로 대체하고, 수정된 `apiVersion`으로 대체한다.
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
`kubectl` 도구는 `pod.yaml`의 내용을 `kind`를 파드(변경되지 않음)로 설정하는 매니페스트로 대체하고, 수정된 `apiVersion`으로 대체한다.
`kubectl` 도구는 `pod.yaml`의 내용을 `kind`를 파드(변경되지 않음, unchanged)로 설정하는 매니페스트로 대체하고, 수정된 `apiVersion`으로 대체한다.

아마도 Pod (unchanged) 가 Pod (unchanged) 코드 형태로 되어야 할 것 같은데 (이 경우, 영문을 그대로 유지)
일단 원문에도 코드 형태로 되어 있지 않으니, 여기서는 간단히 영문을 추가 정보로 제공하는 것으로 진행하면 어떨까 합니다. :)

@k8s-ci-robot k8s-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. area/blog Issues or PRs related to the Kubernetes Blog subproject area/release-eng Issues or PRs related to the Release Engineering subproject language/de Issues or PRs related to German language language/en Issues or PRs related to English language language/es Issues or PRs related to Spanish language language/fr Issues or PRs related to French language language/id Issues or PRs related to Indonesian language language/it Issues or PRs related to Italian language language/ja Issues or PRs related to Japanese language and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 11, 2022
@Kid-Chang Kid-Chang changed the base branch from dev-1.24-ko.2 to dev-1.24-ko.3 August 11, 2022 05:39
@k8s-ci-robot k8s-ci-robot added language/pt Issues or PRs related to Portuguese language language/uk Issues or PRs related to Ukrainian language language/vi Issues or PRs related to Vietnamese language language/zh Issues or PRs related to Chinese language labels Aug 11, 2022
@k8s-ci-robot k8s-ci-robot removed area/release-eng Issues or PRs related to the Release Engineering subproject language/de Issues or PRs related to German language language/en Issues or PRs related to English language language/es Issues or PRs related to Spanish language language/fr Issues or PRs related to French language language/id Issues or PRs related to Indonesian language language/it Issues or PRs related to Italian language language/ja Issues or PRs related to Japanese language language/pl Issues or PRs related to Polish language language/pt Issues or PRs related to Portuguese language language/uk Issues or PRs related to Ukrainian language language/vi Issues or PRs related to Vietnamese language language/zh Issues or PRs related to Chinese language sig/release Categorizes an issue or PR as relevant to SIG Release. labels Aug 11, 2022
[ko] translate 'cluster-upgrade' page v2

[ko] translate 'cluster-upgrade' page
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.

@Kid-Chang 마이너한 수정 사항이 보여서 추가로 의견 드립니다. :)

@seokho-son
Copy link
Member

/label tide/merge-method-squash

@k8s-ci-robot k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Aug 19, 2022
@seokho-son
Copy link
Member

/lgtm
/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 24, 2022
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 2ab434546008e6feb0f5a4af6b63f83ac65fb22c

@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 24, 2022
@k8s-ci-robot k8s-ci-robot merged commit 9a87b4b into kubernetes:dev-1.24-ko.3 Aug 24, 2022
shannonxtreme pushed a commit to shannonxtreme/website that referenced this pull request Sep 22, 2022
* [ko] translate 'cluster-upgrade' page

[ko] translate 'cluster-upgrade' page v2

[ko] translate 'cluster-upgrade' page

* r1-Update content/ko/docs/tasks/administer-cluster/cluster-upgrade.md

* r2-Update content/ko/docs/tasks/administer-cluster/cluster-upgrade.md

Co-authored-by: Seokho Son <[email protected]>
yanrongshi pushed a commit to yanrongshi/website that referenced this pull request Oct 6, 2022
* [ko] translate 'cluster-upgrade' page

[ko] translate 'cluster-upgrade' page v2

[ko] translate 'cluster-upgrade' page

* r1-Update content/ko/docs/tasks/administer-cluster/cluster-upgrade.md

* r2-Update content/ko/docs/tasks/administer-cluster/cluster-upgrade.md

Co-authored-by: Seokho Son <[email protected]>
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. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants