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 files in dev-1.26-ko.1 (M34-M41) #40864

Conversation

jongwooo
Copy link
Member

Signed-off-by: jongwooo [email protected]

Description

Related Issue: #38458

Updated outdated files in content/en/docs/concepts/overview/working-with-objects/.

8 files to be modified

  • M34. content/en/docs/concepts/overview/working-with-objects/annotations.md | 1(+XS) 1(-)
  • M35. content/en/docs/concepts/overview/working-with-objects/common-labels.md | 1(+XS) 0(-)
  • M36. content/en/docs/concepts/overview/working-with-objects/field-selectors.md | 2(+XS) 1(-)
  • M37. content/en/docs/concepts/overview/working-with-objects/finalizers.md | 1(+XS) 1(-)
  • M38. content/en/docs/concepts/overview/working-with-objects/kubernetes-objects.md | 11(+S) 6(-)
  • M39. content/en/docs/concepts/overview/working-with-objects/names.md | 2(+XS) 2(-)
  • M40. content/en/docs/concepts/overview/working-with-objects/namespaces.md | 24(+S) 13(-)
  • M41. content/en/docs/concepts/overview/working-with-objects/object-management.md | 1(+XS) 1(-)

@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. labels Apr 27, 2023
@k8s-ci-robot k8s-ci-robot requested a review from gochist April 27, 2023 01:22
@k8s-ci-robot k8s-ci-robot added the language/ko Issues or PRs related to Korean language label Apr 27, 2023
@k8s-ci-robot k8s-ci-robot requested a review from jmyung April 27, 2023 01:22
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Apr 27, 2023
@netlify
Copy link

netlify bot commented Apr 27, 2023

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 09724a6
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/644b6a43e11b72000848f20e
😎 Deploy Preview https://deploy-preview-40864--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 Author

@jongwooo jongwooo Apr 27, 2023

Choose a reason for hiding this comment

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

25aa28f의 변경사항을 적용하였습니다. 줄바꿈 추가를 제외하고는 이전 번역에서 반영되어 있는 것이라 줄바꿈만 추가하였습니다.

@@ -32,6 +32,26 @@ weight: 30
구별하기 위해 {{< glossary_tooltip text="레이블" term_id="label" >}}을
사용한다.

{{< note >}}
프로덕션 클러스터의 경우, `default` 네임스페이스를 사용하지 _않는_ 것을 고려한다. 대신에, 다른 네임스페이스를 만들어 사용한다.
Copy link
Member Author

Choose a reason for hiding this comment

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

_not__않는_으로 번역하였는데 더 좋은 방법이 있을까요?

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
프로덕션 클러스터의 경우, `default` 네임스페이스를 사용하지 _않는_ 것을 고려한다. 대신에, 다른 네임스페이스를 만들어 사용한다.
프로덕션 클러스터의 경우, `default` 네임스페이스를 _사용하지 않는_ 것을 고려한다. 대신에, 다른 네임스페이스를 만들어 사용한다.

이렇게도 할 수 있겠네요..

둘 다 좋을 것 같습니다. ㅎㅎ

Copy link
Member Author

Choose a reason for hiding this comment

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

@jihoon-seo 알려주신 _사용하지 않는_이라는 표현이 더 명확한 것 같아서 수정하였습니다. 감사합니다!

: 이 네임스페이스는 각 노드와 연관된 [리스](/docs/reference/kubernetes-api/cluster-resources/lease-v1/) 오브젝트를 갖는다. 노드 리스는 kubelet이 [하트비트](/ko/docs/concepts/architecture/nodes/#하트비트)를 보내서 컨트롤 플레인이 노드의 장애를 탐지할 수 있게 한다.

`kube-public`
: 이 네임스페이스는 **모든** 클라이언트(인증되지 않은 클라이언트 포함)가 읽기 권한으로 접근할 수 있다. 이 네임스페이스는 주로 전체 클러스터 중에 공개적으로 드러나서 읽을 수 있는 리소스를 위해 예약되어 있다. 이 네임스페이스의 공개적인 성격은 단지 관례이지 요구 사항은 아니다.
Copy link
Member Author

Choose a reason for hiding this comment

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

8e7f060에서 문장 중 userclient로 변경되어 사용자클라이언트로 변경하였습니다.

@jihoon-seo
Copy link
Member

/assign

Copy link
Member

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@jongwooo 잘 해 주신 것 같습니다. 감사합니다!
/lgtm

@@ -32,6 +32,26 @@ weight: 30
구별하기 위해 {{< glossary_tooltip text="레이블" term_id="label" >}}을
사용한다.

{{< note >}}
프로덕션 클러스터의 경우, `default` 네임스페이스를 사용하지 _않는_ 것을 고려한다. 대신에, 다른 네임스페이스를 만들어 사용한다.
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
프로덕션 클러스터의 경우, `default` 네임스페이스를 사용하지 _않는_ 것을 고려한다. 대신에, 다른 네임스페이스를 만들어 사용한다.
프로덕션 클러스터의 경우, `default` 네임스페이스를 _사용하지 않는_ 것을 고려한다. 대신에, 다른 네임스페이스를 만들어 사용한다.

이렇게도 할 수 있겠네요..

둘 다 좋을 것 같습니다. ㅎㅎ

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

LGTM label has been added.

Git tree hash: 3b36fec586e46cbce35c6684149d1707c077cc47

@jongwooo jongwooo force-pushed the 230427_ko_update_outdated_files branch from 1098c9b to 09724a6 Compare April 28, 2023 06:39
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 28, 2023
@k8s-ci-robot k8s-ci-robot requested a review from jihoon-seo April 28, 2023 06:40
@seokho-son
Copy link
Member

감사합니다!!
/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 Apr 28, 2023
@jihoon-seo
Copy link
Member

감사합니다!
/lgtm

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

LGTM label has been added.

Git tree hash: 99c86e7445b1cf0ae0f31dc0c4e6ea1a6fd94c9b

@k8s-ci-robot k8s-ci-robot merged commit 9230a47 into kubernetes:dev-1.26-ko.1 Apr 28, 2023
@jongwooo jongwooo deleted the 230427_ko_update_outdated_files branch April 28, 2023 10:49
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.

4 participants