Translate tutorials/kubernetes-basics/expose in Korean #27

zer0big · 2018-09-01T09:14:02Z

explore 한글화작업에 대한 PR 요청 드립니다.
/assign @gochist

kubernetes-basics-expose 한글화

…바탕으로 재작업 반영

1. content/ko/docs/home/expose-interactive.html 2. content/ko/docs/home/expose-intro.html

2. 불필요한 조사 (을/를) 제거 3. 불필요한 복수 -> 단수 표기 4. 불필요 2인칭 표현 생략

…ocs-ko into dev-1.12

into dev-1.12

ianychoi · 2018-09-05T00:15:26Z

I have just reviewed the latest commit. Would you make this pull request having just one unique change and divide current pull request into multi pull requests and each pull request has corresponding atomic change OR just reflect comments with current commits & make another pull request(s) for future commits? I am just worried about such situation:

(1) @zer0big made a pull request -> @gochist reviewed -> (2) @zer0big reflects @gochist 's comments

=> such reviewing procedure is definitely fine
=> The reviewer(s) need to review (1) and (2) and I think it is fine.

(1) @zer0big made a pull request -> @gochist reviewed -> (2) @zer0big reflects @gochist 's comments -> (3) @zer0big accumulates more changes into the same pull request -> me reviewed

=> The reviewer(s) need to review all things: (1), (2), and (3)
=> That might lose what one pull request does and also committers & reviewers are on the loop of iterating commits and reviews

ianychoi · 2018-09-04T23:48:01Z

content/ko/docs/tutorials/kubernetes-basics/explore/explore-interactive.html

@@ -21,7 +21,7 @@
        <div class="katacoda">

            <div class="katacoda__alert">
-                To interact with the Terminal, please use the desktop/tablet version
+                터미널과 상화작용하기 위해, 데스크탑/태블릿 버전을 이용한다.


Typo: "상화작용" -> "상호작용"

ianychoi · 2018-09-04T23:52:47Z

content/ko/docs/tutorials/kubernetes-basics/explore/explore-intro.html

-                    more application containers (such as Docker or rkt), and some shared resources for those containers.
-                    Those resources include:</p>
+                <h2>쿠버네티스 파드</h2>
+                <p>모듈 <a href="/docs/tutorials/kubernetes-basics/deploy-intro/">2</a>에서 배포를 생성했을 때, 쿠버네티스는 여러분의 애플리케이션 인스턴스에 <b>파드</b>를 생성했다. 파드는 하나 또는 그 이상의 애플리케이션 컨테이너 (도커 또는 rkt와 같은)들의 그룹을 나타내는 쿠버네티스의 추상적 개념으로 일부는 컨테이너에 대한 자원을 공유한다. 그 자원은 다음을 포함한다:</p>
                <ul>


The sentences are plain-style statements but "여러분" is used for polite expressions with polite-style statements. I would like to suggest to change "당신" or "대상".

ianychoi · 2018-09-04T23:53:37Z

content/ko/docs/tutorials/kubernetes-basics/explore/explore-intro.html

-                    Each Pod is tied to the Node where it is scheduled, and remains there until termination (according
-                    to restart policy) or deletion. In case of a Node failure, identical Pods are scheduled on other
-                    available Nodes in the cluster.</p>
+                <p>파드는 특유한 "로컬호스트" 애플리케이션 모형을 만들어. 상대적으로 밀접하게 결합되어진 상이한 애플리케이션 컨테이너들을 수용할 수 있다. 가령, 파드는 Node.js 앱과 더불어 Node.js 웹서버에 의해 발행되는 데이터를 공급하는 상이한 컨테이너를 함께 수용할 수 있다. 파드 내 컨테이너는 IP 주소, 그리고 포트 스페이스를 공유하고 항상 함께 위치하고 함께 스케쥴링 되고 동일 노드 상의 컨텍스트를 공유하면서 동작한다.</p>


"로컬호스트" is not logical host. "논리 호스트" would be correct.

How about changing "가령" to "예를 들어,"?

ianychoi · 2018-09-04T23:55:50Z

content/ko/docs/tutorials/kubernetes-basics/explore/explore-intro.html

-                    available Nodes in the cluster.</p>
+                <p>파드는 특유한 "로컬호스트" 애플리케이션 모형을 만들어. 상대적으로 밀접하게 결합되어진 상이한 애플리케이션 컨테이너들을 수용할 수 있다. 가령, 파드는 Node.js 앱과 더불어 Node.js 웹서버에 의해 발행되는 데이터를 공급하는 상이한 컨테이너를 함께 수용할 수 있다. 파드 내 컨테이너는 IP 주소, 그리고 포트 스페이스를 공유하고 항상 함께 위치하고 함께 스케쥴링 되고 동일 노드 상의 컨텍스트를 공유하면서 동작한다.</p>
+
+            <p>파드는 쿠버네티스 플랫폼 상에서 최소 단위가 된다. 우리가 쿠버네티스에서 배포를 생성할 때, 그 배포는 컨테이너 내부에서 컨테이너와 함께 파드를 생성한다. 각 파드는 스케쥴 되어진 노드에게 묶여지게 된다. 그리고 (재구동 정책에 따라) 소멸되거나 삭제되기 전까지 그 노드에 유지된다. 노드에 실패가 발생할 경우, 클러스터 내에 가용한 다른 노드들을 대상으로 스케쥴되어진다.</p>


I am a little bit worried about translating atomic to "최소", since "최소" usually implies minimum. Maybe "최소 (atomic)" would be better?

@ianychoi 파드가 쿠버네티스에서 관리/처리되는 최소의 단위 즉, 더 조깰 수 없는 상태라는 의미인지라 "최소"라고 해석했는데, 더 좋은 표현이 있다면 제안 부탁드립니다~^

ianychoi · 2018-09-04T23:57:40Z

content/ko/docs/tutorials/kubernetes-basics/explore/explore-intro.html

-                    applications and their environments. The most common operations can be done with the following
-                    kubectl commands:</p>
+                <h2>kubectl로 문제해결하기</h2>
+                <p>모듈 <a href="/docs/tutorials/kubernetes-basics/deploy-intro/">2</a>에서, 여러분은 Kubectl 커맨드-라인 인터페이스를 사용하였다. 여러분은 배포된 애플리케이션과 그 환경에 대한 정보를 얻기 위해 모듈 3에서도 계속 그것을 사용하게 될 것이다. 가장 보편적인 운용업무는 다음 kubectl 명령어를 이용해 처리될 수 있다:</p>


If there would be no word replacing "여러분은", How about just deleting "여러분은", since deleting the word does not compromise the original meaning?

zer0big · 2018-09-05T01:22:18Z

@ianychoi 리뷰 감사드립니다. 지적해주신 부분 수정완료 했습니다. 단, atomic에 대한 의견은 추가로 검토 부탁 드립니다~^

gochist · 2018-09-05T03:38:53Z

@zer0big
앞으로 커밋 히스토리를 정리해서 PR을 올려주시면 감사드리겠습니다. @ianychoi 님이 지적해주셨듯이 커미터, 리뷰어 모두에게 부담이 됩니다. 본 PR은 관리자에게 편집 권한을 주셨기 때문에 최종 승인단계에서 승인자가 Squash를 대신해드리는 것이 가능하므로 이대로 진행하셔도 괜찮습니다.

zer0big · 2018-09-05T03:45:46Z

@gochist
먼저 의도하지 않게 부담드린 부분 지송합니다 ㅠ. 커밋 히스토리를 정리한다는 부분에 대해 좀 더 구체적인 설명 부탁드려도 될지요~? 혹시라도 여건이 되신다면, 저 같은 PR 수행 초보자들을 위한 가이드 및 Best Practice를 제시해 주셔도 좋을 듯 합니다.

… 상호 등), 2인칭 생략" This reverts commit 45fc251.

k8s-ci-robot · 2018-09-06T03:53:57Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gochist

If they are not already assigned, you can assign the PR to them by writing /assign @gochist in a comment when ready.

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:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

gochist · 2018-09-11T07:05:29Z

#36 으로 계속 진행하겠습니다.
/close

k8s-ci-robot · 2018-09-11T07:05:30Z

@gochist: Closing this PR.

In response to this:

#36 으로 계속 진행하겠습니다.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

zer0big and others added 13 commits August 27, 2018 23:15

Update expose-intro.html

910bc8d

Update expose-intro.html

e8893c6

Update expose-intro.html

e44e16e

Add files via upload

332ab6e

kubernetes-basics-expose 한글화

Merge remote-tracking branch 'upstream/dev-1.12' into dev-1.12

c535ff7

zer0big dev-1.12 브랜치와 kubernetes-docs-ko의 dev-1.12 브랜치 sync 후 원본 파일을 …

8c9c6b5

…바탕으로 재작업 반영

다음 두 파일 제거

d904d1e

1. content/ko/docs/home/expose-interactive.html 2. content/ko/docs/home/expose-intro.html

Review 후 요청사항(by gochist) 반영 완료

9c64f04

1. 추가 요청 사항 반영

9b82224

2. 불필요한 조사 (을/를) 제거 3. 불필요한 복수 -> 단수 표기 4. 불필요 2인칭 표현 생략

Translate tutorials/kubernetes-basics/expose in Korean

fa1f883

Merge branch 'dev-1.12' of https://github.com/kubernetes/kubernetes-d…

1a504c4

…ocs-ko into dev-1.12

Merge branch 'dev-1.12' of https://github.com/zer0big/kubernetes-docs-ko

974ce1b

into dev-1.12

kubernetes-basics - expose 한글화 작업 초안

cce299b

k8s-ci-robot assigned gochist Sep 1, 2018

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 Sep 1, 2018

k8s-ci-robot requested review from claudiajkang and gochist September 1, 2018 09:14

ianychoi reviewed Sep 5, 2018

View reviewed changes

gochist removed their request for review September 6, 2018 00:38

zer0big added 2 commits September 6, 2018 12:44

Review (by ianychoi) 결과 수정 사항 반영 : 오타, 2인칭 생략 등

45fc251

Revert "Review (by ianychoi) 결과 수정 사항 반영 : 오타(로컬 호스트 -> 논리 호스트, 상화 ->…

971aed1

… 상호 등), 2인칭 생략" This reverts commit 45fc251.

gochist mentioned this pull request Sep 10, 2018

Translate tutorials/kubernetes-basics/update in Korean #34

Closed

k8s-ci-robot closed this Sep 11, 2018

cblecker unassigned gochist Oct 15, 2018

zer0big mentioned this pull request Dec 11, 2018

REQUEST: New membership for zerobig kubernetes/org#300

Closed

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Translate tutorials/kubernetes-basics/expose in Korean #27

Translate tutorials/kubernetes-basics/expose in Korean #27

zer0big commented Sep 1, 2018

ianychoi commented Sep 5, 2018

ianychoi Sep 4, 2018

ianychoi Sep 4, 2018

ianychoi Sep 4, 2018

ianychoi Sep 4, 2018

ianychoi Sep 4, 2018

zer0big Sep 5, 2018

ianychoi Sep 4, 2018

zer0big commented Sep 5, 2018 •

edited

Loading

gochist commented Sep 5, 2018 •

edited

Loading

zer0big commented Sep 5, 2018

k8s-ci-robot commented Sep 6, 2018

gochist commented Sep 11, 2018

k8s-ci-robot commented Sep 11, 2018

Translate tutorials/kubernetes-basics/expose in Korean #27

Translate tutorials/kubernetes-basics/expose in Korean #27

Conversation

zer0big commented Sep 1, 2018

ianychoi commented Sep 5, 2018

ianychoi Sep 4, 2018

Choose a reason for hiding this comment

ianychoi Sep 4, 2018

Choose a reason for hiding this comment

ianychoi Sep 4, 2018

Choose a reason for hiding this comment

ianychoi Sep 4, 2018

Choose a reason for hiding this comment

ianychoi Sep 4, 2018

Choose a reason for hiding this comment

zer0big Sep 5, 2018

Choose a reason for hiding this comment

ianychoi Sep 4, 2018

Choose a reason for hiding this comment

zer0big commented Sep 5, 2018 • edited Loading

gochist commented Sep 5, 2018 • edited Loading

zer0big commented Sep 5, 2018

k8s-ci-robot commented Sep 6, 2018

gochist commented Sep 11, 2018

k8s-ci-robot commented Sep 11, 2018

zer0big commented Sep 5, 2018 •

edited

Loading

gochist commented Sep 5, 2018 •

edited

Loading