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

translate assign-pod-node.md #18955

Merged
merged 6 commits into from
Feb 10, 2020
Merged

Conversation

KimMJ
Copy link
Contributor

@KimMJ KimMJ commented Feb 2, 2020

/language ko
/assign @ysyukr

rebase를 잘못해서 부득이하게 새로 PR 생성했습니다.
기존에 작성해주셨던 comment들 반영하였고, 라인 정리도 완료했습니다.
상세한 리뷰 감사합니다..!

@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. sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Feb 2, 2020
@netlify
Copy link

netlify bot commented Feb 2, 2020

Deploy preview for k8s-dev-ko ready!

Built with commit 8d249f9

https://deploy-preview-18955--k8s-dev-ko.netlify.com

Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

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

Here's some feedback.

I speak English well and Korean very little, so I'm commenting about details that might be lost on Korean readers. Please do feel free to skip any suggestion that does not seem helpful.


{{% capture overview %}}

{{< glossary_tooltip text="파드" term_id="pod" >}}를 특정한 {{< glossary_tooltip text="노드(emf)" term_id="node" >}}에서만 동작하도록 하거나,
Copy link
Contributor

Choose a reason for hiding this comment

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

“emf”? I'm puzzled. Maybe this makes sense to people who speak Korean better than I do.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for correction! It was "들" actually.

Comment on lines +74 to +82
* [`kubernetes.io/hostname`](/docs/reference/kubernetes-api/labels-annotations-taints/#kubernetes-io-hostname)
* [`failure-domain.beta.kubernetes.io/zone`](/docs/reference/kubernetes-api/labels-annotations-taints/#failure-domainbetakubernetesiozone)
* [`failure-domain.beta.kubernetes.io/region`](/docs/reference/kubernetes-api/labels-annotations-taints/#failure-domainbetakubernetesioregion)
* [`topology.kubernetes.io/zone`](/docs/reference/kubernetes-api/labels-annotations-taints/#topologykubernetesiozone)
* [`topology.kubernetes.io/region`](/docs/reference/kubernetes-api/labels-annotations-taints/#topologykubernetesiozone)
* [`beta.kubernetes.io/instance-type`](/docs/reference/kubernetes-api/labels-annotations-taints/#beta-kubernetes-io-instance-type)
* [`node.kubernetes.io/instance-type`](/docs/reference/kubernetes-api/labels-annotations-taints/#nodekubernetesioinstance-type)
* [`kubernetes.io/os`](/docs/reference/kubernetes-api/labels-annotations-taints/#kubernetes-io-os)
* [`kubernetes.io/arch`](/docs/reference/kubernetes-api/labels-annotations-taints/#kubernetes-io-arch)
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd consider providing translations for:

  • hostname
  • zone
  • region
  • instance type
  • os
  • arch

@KimMJ what do you think? I expect that readers will have come across all these terms; however, if they have mostly on-premises experience, “instance type” might not be familiar. If one term needs explaining I think it becomes appropriate to explain them all.

Copy link
Member

Choose a reason for hiding this comment

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

@sftim I agree with “instance type” might not be familiar to some readers. However, those difficulties could be applicable to English-native readers as well.
Also, since terms used here are some standard sets of labels which would be used as-is (in English), I hope to keep the @KimMJ 's suggestion.


{{% capture whatsnext %}}

[테인트(Taints)](/docs/concepts/configuration/taint-and-toleration/)는 노드가 특정 파드들을 *쫓아내게* 할 수 있다.
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
[테인트(Taints)](/docs/concepts/configuration/taint-and-toleration/)는 노드가 특정 파드들을 *쫓아내게* 할 수 있다.
[테인트(Taint)](/docs/concepts/configuration/taint-and-toleration/)는 노드가 특정 파드들을 *쫓아내게* 할 수 있다.

?

Copy link
Member

Choose a reason for hiding this comment

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

@KimMJ will change "테인트(Taints)" to "테인트"
Thank you for the correction !


### 1 단계: 노드에 레이블 붙이기

`kubectl get nodes` 를 실행해서 클러스터 노드 이름을 가져온다. 이 중에 레이블을 추가하기 원하는 것 하나를 선택한 다음에 `kubectl label nodes <노드 이름> <레이블 키>=<레이블 값>` 을 실행해서 선택한 노드에 레이블을 추가한다. 예를 들어 노드의 이름이 'kubernetes-foo-node-1.c.a-robinson.internal' 이고, 원하는 레이블이 'disktype=ssd' 라면, `kubectl label nodes ubernetes-foo-node-1.c.a-robinson.internal disktype=ssd` 를 실행한다.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit:

Suggested change
`kubectl get nodes` 를 실행해서 클러스터 노드 이름을 가져온다. 이 중에 레이블을 추가하기 원하는 것 하나를 선택한 다음에 `kubectl label nodes <노드 이름> <레이블 키>=<레이블 값>` 을 실행해서 선택한 노드에 레이블을 추가한다. 예를 들어 노드의 이름이 'kubernetes-foo-node-1.c.a-robinson.internal' 이고, 원하는 레이블이 'disktype=ssd' 라면, `kubectl label nodes ubernetes-foo-node-1.c.a-robinson.internal disktype=ssd` 를 실행한다.
`kubectl get nodes` 를 실행해서 클러스터 노드 이름을 가져온다. 이 중에 레이블을 추가하기 원하는 것 하나를 선택한 다음에 `kubectl label nodes <노드 이름> <레이블 키>=<레이블 값>` 을 실행해서 선택한 노드에 레이블을 추가한다. 예를 들어 노드의 이름이 'kubernetes-foo-node-1.c.a-robinson.internal' 이고, 원하는 레이블이 'disktype=ssd' 라면, `kubectl label nodes kubernetes-foo-node-1.c.a-robinson.internal disktype=ssd` 를 실행한다.

Copy link
Member

Choose a reason for hiding this comment

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

@KimMJ Please update PR according to this @sftim 's comment.

`kubectl get pods -o wide` 를 실행해서 파드가 할당된
"NODE" 를 보면 작동하는지 검증할 수 있다.

## 넘어가기 전에: 내장 노드 레이블들 {#built-in-node-labels}
Copy link
Contributor

Choose a reason for hiding this comment

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

Possibly use a Korean id attribute?

Suggested change
## 넘어가기 전에: 내장 노드 레이블들 {#built-in-node-labels}
## 넘어가기 전에: 내장 노드 레이블들

Copy link
Member

@ysyukr ysyukr Feb 2, 2020

Choose a reason for hiding this comment

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

@sftim
The Korean language team decided on this part at the regular meeting on November 7, 2019.
"Identifiers overridden in English documents are handled in English." - Link
The reason for this is that we talked about the same thing in document #16889.

Copy link
Member

Choose a reason for hiding this comment

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

I agree with the @ysyukr comment which complies with Korean l10n team's consensus.
@sftim Do you mind if I mark this conversation is resolved ?

다른 환경에서는 다른 값일 수 있다.
{{< /note >}}

## 노드 격리(isolation)/제한(restriction)
Copy link
Contributor

Choose a reason for hiding this comment

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

I did wonder whether this would work better as:

Suggested change
## 노드 격리(isolation)/제한(restriction)
## 노드 격리(isolation)/제한(restriction) {#노드-격리-제한}

(and similar below)

What do you think @KimMJ?

Copy link
Member

Choose a reason for hiding this comment

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

@sftim
Do not add identifiers that are not currently in the text.
The addition of arbitrary identifiers is also a very important issue that will cause a full revision to other documents that reference this document.

Copy link
Contributor

Choose a reason for hiding this comment

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

not currently in the text.

My thinking here: as this is a new page, such a proposed change could not break any inbound link.

Copy link
Member

Choose a reason for hiding this comment

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

@sftim Thank you for sharing your idea !
I think
Current phase of Korean l10n focuses on making localized version of English original (upstream) documents. So, we try to sync localized documents with corresponding original documents. For this moment, we sync even errors in original documents.
So, for this phase, additional identifiers for Korean document may not useful even though the additional identifiers do not break any inbound link.

Copy link
Member

@ysyukr ysyukr left a comment

Choose a reason for hiding this comment

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

몇가지 추가로 확인이 필요한 부분을 남겨드립니다.
확인 및 업데이트 부탁드립니다 :)

{{< glossary_tooltip text="파드" term_id="pod" >}}를 특정한 {{< glossary_tooltip text="노드(emf)" term_id="node" >}}에서만 동작하도록 하거나,
특정 노드들을 선호하도록 제한할 수 있다.
이를 수행하는 방법에는 여러 가지가 있으며, 권장되는 접근 방식은 모두
[레이블 셀렉터(label selector)](/ko/docs/concepts/overview/working-with-objects/labels/)를 사용하여 선택한다.
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
[레이블 셀렉터(label selector)](/ko/docs/concepts/overview/working-with-objects/labels/)를 사용하여 선택한다.
[레이블 셀렉터](/ko/docs/concepts/overview/working-with-objects/labels/)를 사용하여 선택한다.


### 0 단계: 사전 준비

이 예시는 쿠버네티스 파드에에 대한 기본적인 이해를 하고 있고 [쿠버네티스 클러스터가 설정](/ko/docs/setup/)되어 있다고 가정한다.
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
이 예시는 쿠버네티스 파드에에 대한 기본적인 이해를 하고 있고 [쿠버네티스 클러스터가 설정](/ko/docs/setup/)되어 있다고 가정한다.
이 예시는 쿠버네티스 파드에 대한 기본적인 이해를 하고 있고 [쿠버네티스 클러스터가 설정](/ko/docs/setup/)되어 있다고 가정한다.

{{< /note >}}

{{< note >}}
파드 안티-어피니티에서는 노드에 일관된 레이블을 지정해야 한다. 즉, 클러스터의 모든 노드는 `topologyKey` 와 매칭되는 적절한 레이블이 가지고 있어야 한다. 일부 또는 모든 노드에 지정된 `topologyKey` 레이블이 없는 경우에는 의도하지 않은 동작을 발생할 수 있다.
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
파드 안티-어피니티에서는 노드에 일관된 레이블을 지정해야 한다. 즉, 클러스터의 모든 노드는 `topologyKey` 와 매칭되는 적절한 레이블이 가지고 있어야 한다. 일부 또는 모든 노드에 지정된 `topologyKey` 레이블이 없는 경우에는 의도하지 않은 동작을 발생할 수 있다.
파드 안티-어피니티에서는 노드에 일관된 레이블을 지정해야 한다. 즉, 클러스터의 모든 노드는 `topologyKey` 와 매칭되는 적절한 레이블을 가지고 있어야 한다. 일부 또는 모든 노드에 지정된 `topologyKey` 레이블이 없는 경우에는 의도하지 않은 동작이 발생할 수 있다.


노드 어피니티와 마찬가지로 현재 파드 어피니티와 안티-어피니티로 부르는 "엄격함" 대 "유연함"의 요구사항을 나타내는 `requiredDuringSchedulingIgnoredDuringExecution` 와
`preferredDuringSchedulingIgnoredDuringExecution` 두 가지 종류가 있다.
앞의 노드 어피니티 섹션의 설명을 보면 본다.
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
앞의 노드 어피니티 섹션의 설명을 보면 본다.
앞의 노드 어피니티 섹션의 설명을 본다.

@KimMJ
Copy link
Contributor Author

KimMJ commented Feb 4, 2020

/assign @seokho-son

@ysyukr
Copy link
Member

ysyukr commented Feb 6, 2020

업데이트 감사합니다.
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 6, 2020
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.

@KimMJ 님. 컨트리뷰션 감사합니다 ~ !
이미 @ysyukr 님께서 리뷰 의견을 주셨기 때문에, 저는 간단히 검토 진행하였습니다.
의견 참고하셔서 업데이트 부탁 드립니다.

Comment on lines 14 to 15
보통 스케줄러가 자동으로 합리적인 배치를 수행하기에 이런 제약 조건은 필요하지 않지만
(예: 노드들에 걸쳐 파드를 분배하거나, 여유 자원이 부족한 노드에 파드를 배치하는 등)
Copy link
Member

Choose a reason for hiding this comment

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

[원문]
Generally such constraints are unnecessary, as the scheduler will automatically do a reasonable placement
(e.g. spread your pods across nodes, not place the pod on a node with insufficient free resources, etc.)
but
원문을 고려하면, 괄호의 내용이 "합리적인 배치"를 수식하므로, 괄호의 위치를 바꾸는 것이 적합해 보입니다.
그리고 제안하신 괄호 안의 내용은 "여유 자원이 부족한 노드에 파드를 배치하는" 은 원문과 반대로 번역된 것으로 보입니다.
아래 제안 참고하시기 바랍니다.

-보통 스케줄러가 자동으로 합리적인 배치를 수행하기에 이런 제약 조건은 필요하지 않지만
-(예: 노드들에 걸쳐 파드를 분배하거나, 여유 자원이 부족한 노드에 파드를 배치하는 등)
+보통 스케줄러가 자동으로 합리적인 배치(예: 노드들에 걸쳐 파드를 분배하거나,
+자원이 부족한 노드에 파드를 배치하지 않는 등)를 수행하기에 이런 제약 조건은 필요하지 않지만

[레이블 셀렉터](/ko/docs/concepts/overview/working-with-objects/labels/)를 사용하여 선택한다.
보통 스케줄러가 자동으로 합리적인 배치를 수행하기에 이런 제약 조건은 필요하지 않지만
(예: 노드들에 걸쳐 파드를 분배하거나, 여유 자원이 부족한 노드에 파드를 배치하는 등)
간혹 파드가 착륙하는 노드에 대해 더 많은 제어를 원할 수 있는 상황이 있다.
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
간혹 파드가 착륙하는 노드에 대해 더 많은 제어를 원할 수 있는 상황이 있다.
간혹 파드가 배치되는 노드에 대해 더 많은 제어를 원할 수 있는 상황이 있다.

원문의 lands 을 직역하면 착륙에 가깝지만, 문장의 자연스러움을 위해 "배치되는" 으로 변경하면 어떨까요? 참고하시기 바랍니다.


## 노드 셀렉터(nodeSelector)

`nodeSelector` 는 가장 간단하고 권장하는 노드 선택 제약 조건의 형태이다.
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
`nodeSelector` 는 가장 간단하고 권장하는 노드 선택 제약 조건의 형태이다.
`nodeSelector` 는 가장 간단하고 권장되는 노드 선택 제약 조건의 형태이다.

조금 자연스럽게 변경해 보았습니다. 참고하시기 바랍니다.

`nodeSelector` 는 가장 간단하고 권장하는 노드 선택 제약 조건의 형태이다.
`nodeSelector` 는 PodSpec의 필드이다. 이는 키-값 쌍의 매핑으로 지정한다. 파드가 노드에서 동작할 수 있으려면,
노드는 키-값의 쌍으로 표시되는 레이블을 각자 가지고 있어야 한다(이는 추가 레이블을 가지고 있을 수 있다).
대부분의 일반적인 사용은 하나의 키-값 쌍이다.
Copy link
Member

Choose a reason for hiding this comment

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

The most common usage is one key-value pair.

Suggested change
대부분의 일반적인 사용은 하나의 키-값 쌍이다.
일반적으로 하나의 키-값 쌍이 사용된다.

직역된 문장을 조금 자연스럽게 수정해 보았습니다. 참고하시기 바랍니다.


### 2 단계: 파드 설정에 nodeSelector 필드 추가하기

실행하고자 하는 파드의 설정 파일을 가져오고, 이처럼 nodeSelector 섹션을 추가한다. 예를 들어 이것이 파드 설정이라면:
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
실행하고자 하는 파드의 설정 파일을 가져오고, 이처럼 nodeSelector 섹션을 추가한다. 예를 들어 이것이 파드 설정이라면:
실행하고자 하는 파드의 설정 파일을 가져오고, 이처럼 nodeSelector 섹션을 추가한다. 예를 들어 이것이 파드 설정이라면,

3. `preferredDuringSchedulingIgnoredDuringExecution` 파드 안티-어피니티는 빈 `topologyKey` 를 "all topology"("all topology"는 현재 `kubernetes.io/hostname`, `failure-domain.beta.kubernetes.io/zone` 그리고 `failure-domain.beta.kubernetes.io/region` 의 조합으로 제한된다).
4. 위의 경우를 제외하고, `topologyKey` 는 적법한 어느 레이블-키도 가능하다.

`labelSelector` 와 `topologyKey` 외에도 `labelSelector` 와 일치해야하는 네임스페이스 목록 `namespaces` 를
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
`labelSelector``topologyKey` 외에도 `labelSelector`일치해야하는 네임스페이스 목록 `namespaces`
`labelSelector``topologyKey` 외에도 `labelSelector`일치해야 하는 네임스페이스 목록 `namespaces`


세 개의 노드가 있는 클러스터에서 웹 애플리케이션에는 redis와 같은 인-메모리 캐시가 있다. 웹 서버가 가능한 캐시와 함께 위치하기를 원한다.

다음은 세 개의 레플리카와 셀렉터 레이블이 `app=store` 가 있는 간단한 redis 디플로이먼트의 yaml 스니펫이다. 디플로이먼트에는 스케줄러가 단일 노드에서 레플리카를 함꼐 배치하지 않도록 `PodAntiAffinity` 가 구성되어있다.
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
다음은 세 개의 레플리카와 셀렉터 레이블이 `app=store` 가 있는 간단한 redis 디플로이먼트의 yaml 스니펫이다. 디플로이먼트에는 스케줄러가 단일 노드에서 레플리카를 함꼐 배치하지 않도록 `PodAntiAffinity`구성되어있다.
다음은 세 개의 레플리카와 셀렉터 레이블이 `app=store` 가 있는 간단한 redis 디플로이먼트의 yaml 스니펫이다. 디플로이먼트에는 스케줄러가 단일 노드에서 레플리카를 함께 배치하지 않도록 `PodAntiAffinity`구성되어 있다.

image: redis:3.2-alpine
```

아래 yaml 스니펫의 웹서버 디플로이먼트는 `podAntiAffinity` 와 `podAffinity` 설정을 가지고 있다. 이렇게 하면 스케줄러에 모든 레플리카는 셀렉터 레이블이 `app=store` 인 파드와 함꼐 위치해야 한다. 또한 각 웹 서버 레플리카가 단일 노드의 같은 위치에 있지 않도록 한다.
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
아래 yaml 스니펫의 웹서버 디플로이먼트는 `podAntiAffinity``podAffinity` 설정을 가지고 있다. 이렇게 하면 스케줄러에 모든 레플리카는 셀렉터 레이블이 `app=store` 인 파드와 함꼐 위치해야 한다. 또한 각 웹 서버 레플리카가 단일 노드의 같은 위치에 있지 않도록 한다.
아래 yaml 스니펫의 웹서버 디플로이먼트는 `podAntiAffinity``podAffinity` 설정을 가지고 있다. 이렇게 하면 스케줄러에 모든 레플리카는 셀렉터 레이블이 `app=store` 인 파드와 함께 위치해야 한다. 또한 각 웹 서버 레플리카가 단일 노드의 같은 위치에 있지 않도록 한다.

```
kubectl get pods -o wide
```
출력은 다음과 유사할 것이다:
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
출력은 다음과 유사할 것이다:
출력은 다음과 유사할 것이다.


{{% capture whatsnext %}}

[테인트(Taints)](/docs/concepts/configuration/taint-and-toleration/)는 노드가 특정 파드들을 *쫓아내게* 할 수 있다.
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
[테인트(Taints)](/docs/concepts/configuration/taint-and-toleration/)는 노드가 특정 파드들을 *쫓아내게* 할 수 있다.
[테인트](/docs/concepts/configuration/taint-and-toleration/)는 노드가 특정 파드들을 *쫓아내게* 할 수 있다.

Since we inform the original English once for a word in a document, we don't need to provide English word here again.
In 153 line, Taint was firstly translated to "테인트(taint)" already.
I think this comment will resolve @sftim 's correction as well.

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 9, 2020
@KimMJ
Copy link
Contributor Author

KimMJ commented Feb 9, 2020

리뷰 업데이트 완료했습니다!

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.

@KimMJ 업데이트 감사합니다 👍
페이지에 오류 문자가 포함되어 있어, 수정 요청 드립니다.

이를 수행하는 방법에는 여러 가지가 있으며, 권장되는 접근 방식은 모두
[레이블 셀렉터](/ko/docs/concepts/overview/working-with-objects/labels/)를 사용하여 선택한다.
보통 스케줄러가 자동으로 합리적인 배치(예: 노드들에 걸쳐 파드를 분배하거나,
자원이 부족한 노드에 파드를 배치하지 않는 등)를 수행하기에 이런 제약 조건은 필요하지 않지만
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
자원이 부족한 노드에 파드를 배치하지 않는 �등)를 수행하기에 이런 제약 조건은 필요하지 않지만
자원이 부족한 노드에 파드를 배치하지 않는 등)를 수행하기에 이런 제약 조건은 필요하지 않지만

에티터 등을 통해서 오류 문자가 포함된 것 같습니다. "�" 삭제가 필요합니다.
참고) 이러한 오류는 페이지 프리뷰를 보시면 확인이 쉽습니다. 페이지 프리뷰는 netlify 봇이 PR에 코멘트로 링크를 남깁니다 ^^ https://deploy-preview-18955--k8s-dev-ko.netlify.com/ko/docs/concepts/configuration/assign-pod-node/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

맥북-vscode로 작업 시 자꾸 저런 문자가 저장할 때 생기네요..
확인할때 유의하도록 하겠습니다!

`nodeSelector` 는 가장 간단하고 권장되는 노드 선택 제약 조건의 형태이다.
`nodeSelector` 는 PodSpec의 필드이다. 이는 키-값 쌍의 매핑으로 지정한다. 파드가 노드에서 동작할 수 있으려면,
노드는 키-값의 쌍으로 표시되는 레이블을 각자 가지고 있어야 한다(이는 추가 레이블을 가지고 있을 수 있다).
일반적으로 하나의 키-값 쌍이 사용된다.
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
일반적으로 �하나의 키-값 쌍이 사용된다.
일반적으로 하나의 키-값 쌍이 사용된다.

여기에 현재 `requiredDuringSchedulingIgnoredDuringExecution` 와 `preferredDuringSchedulingIgnoredDuringExecution` 로 부르는
두 가지 종류의 노드 어피니티가 있다. 전자는 파드가 노드에 스케줄 되도록 *반드시*
규칙을 만족해야 하는 것(`nodeSelector` 와 같으나 보다 표현적인 구문을 사용해서)을 지정하고,
후자는 스케줄러가 시도하려고는 하지만, 보증하지 않는 *선호(preferences)*를 지정한다는 점에서
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
후자는 스케줄러가 시도하려고는 하지만, 보증하지 않는 *선호(preferences)*를 지정한다는 점에서
후자는 스케줄러가 시도하려고는 하지만, 보증하지 않는 *선호(preferences)* 를 지정한다는 점에서

* xxx * 마크다운이 제대로 빌드되지 않는 문제가 있습니다. * 뒤에 띄어쓰기 추가가 필요합니다.

@seokho-son
Copy link
Member

I think this PR can be approved :)
Thank you for the contribution and reviews! @KimMJ @ysyukr @sftim

/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 Feb 10, 2020
@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 Feb 10, 2020
@k8s-ci-robot k8s-ci-robot merged commit 32f172b into kubernetes:dev-1.17-ko.4 Feb 10, 2020
gochist pushed a commit that referenced this pull request Feb 15, 2020
* translate assign-pod-node.md (#18955)
* correct the misspelling (#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>
k8s-ci-robot pushed a commit that referenced this pull request Feb 15, 2020
* translate assign-pod-node.md (#18955)
* correct the misspelling (#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[email protected]>
zacharysarah pushed a commit to zacharysarah/website that referenced this pull request Feb 15, 2020
* translate assign-pod-node.md (kubernetes#18955)
* correct the misspelling (kubernetes#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (kubernetes#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[email protected]>
kbhawkey pushed a commit to kbhawkey/website that referenced this pull request Feb 18, 2020
* translate assign-pod-node.md (kubernetes#18955)
* correct the misspelling (kubernetes#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (kubernetes#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[email protected]>
kbhawkey pushed a commit to kbhawkey/website that referenced this pull request Feb 19, 2020
* translate assign-pod-node.md (kubernetes#18955)
* correct the misspelling (kubernetes#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (kubernetes#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[email protected]>
zacharysarah pushed a commit to zacharysarah/website that referenced this pull request Feb 21, 2020
* translate assign-pod-node.md (kubernetes#18955)
* correct the misspelling (kubernetes#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (kubernetes#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[email protected]>
wawa0210 pushed a commit to wawa0210/website that referenced this pull request Mar 2, 2020
* translate assign-pod-node.md (kubernetes#18955)
* correct the misspelling (kubernetes#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (kubernetes#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[email protected]>
VineethReddy02 pushed a commit to VineethReddy02/website that referenced this pull request Mar 22, 2020
* translate assign-pod-node.md (kubernetes#18955)
* correct the misspelling (kubernetes#19063)
* Update to Outdated files in dev-1.17-ko.4 branch. (kubernetes#19002)

Co-Authored-By: KimMJ <[email protected]>
Co-Authored-By: forybm <[email protected]>
Co-Authored-By: Yuk, Yongsu <[email protected]>

Co-authored-by: KimMJ <[email protected]>
Co-authored-by: forybm <[email protected]>
Co-authored-by: Yuk, Yongsu <[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/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