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 (M83-M97) #38709

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

bconfiden2
Copy link
Member

15 files to be modified

  1. M83. content/en/docs/concepts/workloads/_index.md | 1(+XS) 1(-)
  2. M84. content/en/docs/concepts/workloads/controllers/cron-jobs.md | 8(+XS) 7(-)
  3. M85. content/en/docs/concepts/workloads/controllers/daemonset.md | 1(+XS) 2(-)
  4. M86. content/en/docs/concepts/workloads/controllers/deployment.md | 2(+XS) 2(-)
  5. M87. content/en/docs/concepts/workloads/controllers/job.md | 61(+M) 53(-)
  6. M88. content/en/docs/concepts/workloads/controllers/replicaset.md | 7(+XS) 0(-)
  7. M89. content/en/docs/concepts/workloads/controllers/replicationcontroller.md | 2(+XS) 8(-)
  8. M90. content/en/docs/concepts/workloads/controllers/statefulset.md | 23(+S) 8(-)
  9. M91. content/en/docs/concepts/workloads/pods/_index.md | 2(+XS) 4(-)
  10. M92. content/en/docs/concepts/workloads/pods/disruptions.md | 15(+S) 3(-)
  11. M93. content/en/docs/concepts/workloads/pods/downward-api.md | 1(+XS) 0(-)
  12. M94. content/en/docs/concepts/workloads/pods/ephemeral-containers.md | 4(+XS) 0(-)
  13. M95. content/en/docs/concepts/workloads/pods/init-containers.md | 2(+XS) 2(-)
  14. M96. content/en/docs/concepts/workloads/pods/pod-lifecycle.md | 17(+S) 3(-)
  15. M97. content/en/docs/concepts/workloads/pods/user-namespaces.md | 3(+XS) 3(-)

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 31, 2022
@k8s-ci-robot k8s-ci-robot requested a review from pjhwa December 31, 2022 08:19
@k8s-ci-robot k8s-ci-robot added the language/ko Issues or PRs related to Korean language label Dec 31, 2022
@k8s-ci-robot k8s-ci-robot added 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. labels Dec 31, 2022
@netlify
Copy link

netlify bot commented Dec 31, 2022

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit bbd772c
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/641906a87ece580008e08b76
😎 Deploy Preview https://deploy-preview-38709--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

/assign @seokho-son

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.

@bconfiden2 기여 감사합니다! :)

꼼꼼하게 라인까지 잘 정리해주셨네요.
사소한 의견들이 있어서, 의견 남겨두었습니다.

적절하게 반영해주시면 되겠습니다. (커밋은 그냥 amend 해주셔도 괜찮을 것 같습니다.)


중요한 것은 만약 `startingDeadlineSeconds` 필드가 설정이 되면(`nil` 이 아닌 값으로), 컨트롤러는 마지막 일정부터 지금까지 대신 `startingDeadlineSeconds` 값에서 몇 개의 잡이 누락되었는지 카운팅한다. 예를 들면, `startingDeadlineSeconds` 가 `200` 이면, 컨트롤러는 최근 200초 내 몇 개의 잡이 누락되었는지 카운팅한다.

크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들면, `concurrencyPolicy` 가 `Forbid` 로 설정되었고, 크론잡이 이전 일정이 스케줄되어 여전히 시도하고 있을 때, 그 때 누락되었다고 판단한다.
크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들어 `concurrencyPolicy`가 `Forbid` 로 설정된 상태에서, 이전 잡을 여전히 수행하던 중 크론잡이 스케줄되는 경우에도 잡 실행에 실패하였다고 판단한다.
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
크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들어 `concurrencyPolicy``Forbid`설정된 상태에서, 이전 잡을 여전히 수행하던 중 크론잡이 스케줄되는 경우에도 잡 실행에 실패하였다고 판단한다.
크론잡은 정해진 일정에 생성을 실패하면 누락(missed)된 것으로 집계된다. 예를 들어 `concurrencyPolicy``Forbid`설정되어 있고 이전 크론잡이 여전히 실행 중인 상태에서, 크론잡이 일정에 따라 시도되면 해당 크론잡의 실패로 인해 누락된 것으로 집계될 것이다.

A CronJob is counted as missed if it has failed to be created at its scheduled time. For example, if concurrencyPolicy is set to Forbid and a CronJob was attempted to be scheduled when there was a previous schedule still running, then it would count as missed.

Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들어 `concurrencyPolicy``Forbid`설정된 상태에서, 이전 잡을 여전히 수행하던 중 크론잡이 스케줄되는 경우에도 잡 실행에 실패하였다고 판단한다.
크론잡은 정해진 시간에 생성되는데 실패하면 누락(missed)된 것으로 집계된다. 예를 들어 `concurrencyPolicy``Forbid`설정되어 있고 이전 크론잡이 여전히 실행 중인 상태라면, 크론잡은 일정에 따라 시도되었다가 생성을 실패하고 누락된 것으로 집계될 것이다.

리뷰 반영하여서 다시 다듬어 보았습니다 !!

content/ko/docs/concepts/workloads/controllers/job.md Outdated Show resolved Hide resolved
content/ko/docs/concepts/workloads/controllers/job.md Outdated Show resolved Hide resolved
content/ko/docs/concepts/workloads/pods/disruptions.md Outdated Show resolved Hide resolved
content/ko/docs/concepts/workloads/pods/pod-lifecycle.md Outdated Show resolved Hide resolved
content/ko/docs/concepts/workloads/pods/pod-lifecycle.md Outdated Show resolved Hide resolved
Copy link
Member Author

@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.

꼼꼼한 리뷰 감사합니다!!


중요한 것은 만약 `startingDeadlineSeconds` 필드가 설정이 되면(`nil` 이 아닌 값으로), 컨트롤러는 마지막 일정부터 지금까지 대신 `startingDeadlineSeconds` 값에서 몇 개의 잡이 누락되었는지 카운팅한다. 예를 들면, `startingDeadlineSeconds` 가 `200` 이면, 컨트롤러는 최근 200초 내 몇 개의 잡이 누락되었는지 카운팅한다.

크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들면, `concurrencyPolicy` 가 `Forbid` 로 설정되었고, 크론잡이 이전 일정이 스케줄되어 여전히 시도하고 있을 때, 그 때 누락되었다고 판단한다.
크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들어 `concurrencyPolicy`가 `Forbid` 로 설정된 상태에서, 이전 잡을 여전히 수행하던 중 크론잡이 스케줄되는 경우에도 잡 실행에 실패하였다고 판단한다.
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
크론잡은 정해진 일정에 잡 실행을 실패하면 놓쳤다고 카운팅된다. 예를 들어 `concurrencyPolicy``Forbid`설정된 상태에서, 이전 잡을 여전히 수행하던 중 크론잡이 스케줄되는 경우에도 잡 실행에 실패하였다고 판단한다.
크론잡은 정해진 시간에 생성되는데 실패하면 누락(missed)된 것으로 집계된다. 예를 들어 `concurrencyPolicy``Forbid`설정되어 있고 이전 크론잡이 여전히 실행 중인 상태라면, 크론잡은 일정에 따라 시도되었다가 생성을 실패하고 누락된 것으로 집계될 것이다.

리뷰 반영하여서 다시 다듬어 보았습니다 !!

@seokho-son
Copy link
Member

/retitle [ko] Update outdated files in dev-1.26-ko.1 (M83-M97)

@k8s-ci-robot k8s-ci-robot changed the title Update outdated files in dev-1.26-ko.1 (M83-M97) [ko] Update outdated files in dev-1.26-ko.1 (M83-M97) Mar 21, 2023
@seokho-son
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 Mar 21, 2023
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 6021c2235c93065d440e786d881ca99ec14debcd

@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 Mar 21, 2023
@k8s-ci-robot k8s-ci-robot merged commit 498f9e3 into kubernetes:dev-1.26-ko.1 Mar 21, 2023
@bconfiden2 bconfiden2 deleted the outdated-1231 branch March 21, 2023 03:06
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.

3 participants