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 docs/tasks/administer-cluster/limit-storage-consumption into Korean #36147

Conversation

dewble
Copy link
Contributor

@dewble dewble commented Aug 21, 2022

fix #35200

@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 Aug 21, 2022
@k8s-ci-robot k8s-ci-robot added the language/ko Issues or PRs related to Korean language label Aug 21, 2022
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Aug 21, 2022
@dewble dewble changed the base branch from main to dev-1.24-ko.3 August 21, 2022 16:15
@dewble dewble changed the title Translate limit storage consumption into Korean [ko]Translate limit storage consumption into Korean Aug 21, 2022
@netlify
Copy link

netlify bot commented Aug 21, 2022

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit ea31add
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/63025957533ddb00089e1080
😎 Deploy Preview https://deploy-preview-36147--kubernetes-io-main-staging.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.

한영병기의 경우 문서 전반에 걸쳐 되어있길래 하나하나 달지는 않았습니다..! 혹시 한영병기가 있는 것이 좋다고 보시면 그대로 두셔도 괜찮을 것 같아요!!
그 외에 몇몇 부분을 좀 더 풀어서 정리해봤는데, 같이 얘기 나눠보면 좋겠습니다 😃 😃

@bart2001
Copy link
Contributor

한글화 용어집에 따르면
범위 제한(LimitRange)리밋레인지(LimitRange)로 표기되어서 요렇게 일괄 수정 제안드립니다.

@dewble dewble changed the title [ko]Translate limit storage consumption into Korean [ko]Translate docs/tasks/administer-cluster/limit-storage-consumption into Korean Sep 6, 2022
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.

@dewble 기여 감사합니다!
기존 리뷰 의견들을 모두 고려하여,
전반적으로 리뷰하였습니다. 참고하셔서 적절히 업데이트 부탁 드립니다.
(추가적으로 업스트림 영문에 제안 사항도 있으니 살펴보시길.. ^^)

@bart2001 @bconfiden2 리뷰 의견에 감사드립니다!!


범위 제한(LimitRange)을 지정하면 요청된 스토리지 양을 제한할 수 있으며
리소스쿼터(ResourceQuota)는 네임스페이스(Namespace)에 클레임(claim)수와 누적 스토리지 용량을 효과적으로 제한할 수 있다.
클러스터 관리자는 어느 프로젝트도 할당량을 초과할 위험 없이 클러스터의 스토리지 예산을 계획할 수 있다.
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 allows a cluster-admin to plan their
cluster's storage budget without risk of any one project going over their allotment.

The allows 가 아니라 This allows 또는 It allows 인데 오타가 아닌가 싶습니다.
@dewble 영문 쪽에도 수정 기여를 해보시면 어떨까요? ㅎㅎ
(PR을 upstream/main 에 올리면 됩니다.)

@seokho-son
Copy link
Member

/assign @seokho-son @bconfiden2 @bart2001

@dewble dewble force-pushed the ko-translate-limit-storage-consumption branch from ea31add to 2c6f20f Compare September 12, 2022 08:23
@netlify
Copy link

netlify bot commented Sep 12, 2022

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 2c6f20f
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/631eec9b09121000085e104c
😎 Deploy Preview https://deploy-preview-36147--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.

@seokho-son
Copy link
Member

Thanks @dewble !

/lgtm

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

LGTM label has been added.

Git tree hash: 84a6405f3ea52a7a46d1302592a48bf4a4a7e763

@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 Sep 12, 2022
@k8s-ci-robot k8s-ci-robot merged commit e05f790 into kubernetes:dev-1.24-ko.3 Sep 12, 2022
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.

5 participants