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 health-checks.md into Korean #25342

Merged
merged 1 commit into from
Dec 3, 2020

Conversation

jm1223kim
Copy link
Contributor

from #24982

@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


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. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Dec 2, 2020
@k8s-ci-robot
Copy link
Contributor

Welcome @jm1223kim!

It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/website has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 2, 2020
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Dec 2, 2020
@netlify
Copy link

netlify bot commented Dec 2, 2020

Deploy preview for k8s-dev-ko ready!

Built with commit b414466

https://deploy-preview-25342--k8s-dev-ko.netlify.app

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Dec 2, 2020
@pjhwa
Copy link
Contributor

pjhwa commented Dec 2, 2020

/assign @pjhwa

@pjhwa
Copy link
Contributor

pjhwa commented Dec 2, 2020

Preview page: https://deploy-preview-25342--k8s-dev-ko.netlify.app/ko/docs/reference/using-api/health-checks/

Copy link
Contributor

@pjhwa pjhwa left a comment

Choose a reason for hiding this comment

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

@jm1223kim 첫 번째 컨트리뷰션 감사합니다!!!

몇 가지 작은 의견을 드립니다. 확인하시고, 반영 부탁드립니다.
반영 후 커밋들을 하나로 합치는 스쿼시 작업도 부탁드립니다.

Comment on lines 3 to 4
reviewers:
-
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
reviewers:
-


<!-- overview -->
쿠버네티스 {{< glossary_tooltip term_id="kube-apiserver" text="API 서버" >}}는 현재 상태를 나타내는 API 엔드포인트를 제공한다.
이 페이지에서는 API 엔드포인트들에 대해 설명하고 이를 사용하는 방법에 대해 설명한다.
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
이 페이지에서는 API 엔드포인트들에 대해 설명하고 이를 사용하는 방법에 대해 설명한다.
이 페이지에서는 API 엔드포인트들에 대해 설명하고 이를 사용하는 방법을 다룬다.
  • 좀 더 자연스럽게 수정해봤습니다.


<!-- body -->

## 헬스(health)를 위한 API 엔드포인트
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
## 헬스(health)를 위한 API 엔드포인트
## 헬스를 위한 API 엔드포인트
  • 제목에서 한영병기했기 때문에 생략해도 좋을 것 같습니다.


## 헬스(health)를 위한 API 엔드포인트

쿠버네티스 API 서버는 현재 상태를 나타내는 3가지 API 엔드포인트(`healthz`, `livez` 와 `readyz`)를 제공한다.
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
쿠버네티스 API 서버는 현재 상태를 나타내는 3가지 API 엔드포인트(`healthz`, `livez``readyz`)를 제공한다.
쿠버네티스 API 서버는 현재 상태를 나타내는 세 가지 API 엔드포인트(`healthz`, `livez``readyz`)를 제공한다.
  • 좀 더 자연스럽게 수정해봤습니다.

## 헬스(health)를 위한 API 엔드포인트

쿠버네티스 API 서버는 현재 상태를 나타내는 3가지 API 엔드포인트(`healthz`, `livez` 와 `readyz`)를 제공한다.
`healthz` 엔드포인트는 사용 중단(deprecated) 됐으며 (Kubernetes v1.16 버전이후), 대신 보다 구체적인 `livez` 와 `readyz` 엔드포인트를 사용해야 한다.
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
`healthz` 엔드포인트는 사용 중단(deprecated) 됐으며 (Kubernetes v1.16 버전이후), 대신 보다 구체적인 `livez``readyz` 엔드포인트를 사용해야 한다.
`healthz` 엔드포인트는 사용 중단(deprecated)됐으며(쿠버네티스 v1.16 버전 이후), 대신 보다 구체적인 `livez``readyz` 엔드포인트를 사용해야 한다.
  • 띄어쓰기, 용어 한글화를 수정했습니다.

curl -k 'https://localhost:6443/readyz?verbose&exclude=etcd'
```

출력에서 etcd 체크가 제외된 것을 보여준다:
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
출력에서 etcd 체크가 제외된 것을 보여준다:
출력에서 etcd 체크가 제외된 것을 보여준다.

[+]shutdown ok
healthz check passed

## 개별 헬스(health) 체크
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
## 개별 헬스(health) 체크
## 개별 헬스 체크
  • 한영병기는 생략해도 좋을 것 같습니다.


{{< feature-state state="alpha" >}}

각 http 엔드포인트를 개별적으로 노출하여 헬스 체크가 가능하다.
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
각 http 엔드포인트를 개별적으로 노출하여 헬스 체크가 가능하다.
개별 헬스 체크는 http 엔드포인트를 노출하고 개별적으로 체크가 가능하다.
  • 원문에 좀 더 가깝게 수정해봤습니다.

Each individual health check exposes an http endpoint and could can be checked individually.

각 http 엔드포인트를 개별적으로 노출하여 헬스 체크가 가능하다.
개별 체크를 위한 스키마는 `/livez/<healthcheck-name>` 이고, 여기서 `livez` 와 `readyz` 는 API 서버의 활성 상태 또는 준비 상태인지를 확인할 때 사용한다.
`<healthcheck-name>` 경로 위에 `verbose` 옵션을 사용할 때 발견될 수 있고, `[+]` 와 `ok` 사이의 경로를 사용한다.
이러한 개별 헬스 체크는 머신에서 사용되서는 안되며, 운영자가 시스템의 현재 상태를 디버그 하는데 유용하다:
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
이러한 개별 헬스 체크는 머신에서 사용되서는 안되며, 운영자가 시스템의 현재 상태를 디버그 하는데 유용하다:
이러한 개별 헬스 체크는 머신에서 사용되서는 안되며, 운영자가 시스템의 현재 상태를 디버깅하는데 유용하다.


각 http 엔드포인트를 개별적으로 노출하여 헬스 체크가 가능하다.
개별 체크를 위한 스키마는 `/livez/<healthcheck-name>` 이고, 여기서 `livez` 와 `readyz` 는 API 서버의 활성 상태 또는 준비 상태인지를 확인할 때 사용한다.
`<healthcheck-name>` 경로 위에 `verbose` 옵션을 사용할 때 발견될 수 있고, `[+]` 와 `ok` 사이의 경로를 사용한다.
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
`<healthcheck-name>` 경로 위에 `verbose` 옵션을 사용할 때 발견될 수 있고, `[+]``ok` 사이의 경로를 사용한다.
`<healthcheck-name>` 경로 위에서 설명한 `verbose` 플래그를 사용해서 찾을 수 있고, `[+]``ok` 사이의 경로를 사용한다.
  • 앞서 설명한 'verbose' 플래그를 의미하는 것 같습니다.

The <healthcheck-name> path can be discovered using the verbose flag from above and take the path between [+] and ok.

@jm1223kim jm1223kim force-pushed the translate-24982 branch 2 times, most recently from 7610400 to bba4e28 Compare December 2, 2020 08:53
@jm1223kim
Copy link
Contributor Author

@pjhwa 의견 주셔서 감사합니다. 모두 반영 완료했습니다.

Copy link
Contributor

@pjhwa pjhwa left a comment

Choose a reason for hiding this comment

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

@jm1223kim 감사합니다!
이전 리뷰에서 놓친 것이 있어 추가 의견 드립니다.

kubectl get --raw='/readyz?verbose'
```

출력은 다음과 같다:
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
출력은 다음과 같다:
출력은 다음과 같다.
  • 콜론(:) 사용을 수정했습니다.

@jm1223kim
Copy link
Contributor Author

@pjhwa 의견 주신 사항 다시 반영하여 올렸습니다. 감사합니다.

`livez` 엔드포인트는 `--livez-grace-period` [플래그](/docs/reference/command-line-tools-reference/kube-apiserver) 옵션을 사용하여 시작 대기 시간을 지정할 수 있다.
`/readyz` 엔드포인트는 `--shutdown-delay-duration` [플래그](/docs/reference/command-line-tools-reference/kube-apiserver) 옵션을 사용하여 정상적(graceful)으로 셧다운할 수 있다.
API 서버의 `health`/`livez`/`readyz` 를 사용하는 머신은 HTTP 상태 코드에 의존해야 한다.
상태 코드200 은 호출된 엔드포인트에 따라 API 서버의 `healthy`/`live`/`ready` 상태를 나타낸다.
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
상태 코드200 은 호출된 엔드포인트에 따라 API 서버의 `healthy`/`live`/`ready` 상태를 나타낸다.
상태 코드 200은 호출된 엔드포인트에 따라 API 서버의 `healthy`/`live`/`ready` 상태를 나타낸다.
  • 이 부분도 추가 수정해주시겠습니까? 잘못 고쳐진 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네, 수정 완료했습니다.

@pjhwa
Copy link
Contributor

pjhwa commented Dec 2, 2020

@jm1223kim 감사합니다!

/lgtm

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

LGTM label has been added.

Git tree hash: 651e8abe749a82bc84ce7477d52d437be5d00f82

@seokho-son
Copy link
Member

@jm1223kim 기여 감사합니다! :)
기여하신 내용은 dev-1.19-ko.6 브랜치가 master 브랜치에 머지되는 시점에 k8s.io 에서 확인 가능합니다!

/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 Dec 3, 2020
@k8s-ci-robot k8s-ci-robot merged commit 057b086 into kubernetes:dev-1.19-ko.6 Dec 3, 2020
@jm1223kim
Copy link
Contributor Author

@seokho-son 감사합니다. :)

seokho-son added a commit that referenced this pull request Dec 6, 2020
- fix Korean translation of 'object' (#25370)
- Update outdated files in the dev-1.19-ko.6 branch (2) (#25403)
- Translate setup/release/version-skew-policy.md in Korean (#25345)
- Translate tutorials/configuration/configure-java-microservice/configure-java-microservice/ into Korean (#24981)
- Translate health-checks.md into Korean (#25342)
- Translate reference/glossary/api-group.md in Korean (#25321)
- Translate reference/glossary/object.md in Korean (#25318)
- Translate tutorials/configuration/configure-java-microservice/configure-java-microservice-interactive/ into Korean (#24984)
- switch New York Times case study to use youtube shortcode (ko) (#25287)

Co-authored-by: seokho-son <shsongist@gmail.com>
Co-authored-by: Jerry Park <jaehwa@gmail.com>
Co-authored-by: PyungHo Yoon <learder@gmail.com>
Co-authored-by: mylovepooh <jun8414@gmail.com>
Co-authored-by: jmkim <jm1223.kim@samsung.com>
Co-authored-by: santachopa <santachopa@naver.com>
Co-authored-by: Arhell <arhell333@gmail.com>
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.

4 participants