Skip to content

[ko] Update /ko/concepts/workloads/controllers/daemonset.md #52944

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[ko] Update /ko/concepts/workloads/controllers/daemonset.md #52944

wants to merge 2 commits into from
+86 −64

Conversation

@Eundms
Copy link
Member

@Eundms Eundms commented Oct 25, 2025

Description

  • Sync /ko/docs/concepts/workloads/controllers/daemonset.md .

Issue

Closes: #52553

@k8s-ci-robot k8s-ci-robot added area/localization General issues or PRs related to localization cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 25, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign jihoon-seo for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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 language/ko Issues or PRs related to Korean language 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 Oct 25, 2025
@netlify
Copy link

netlify bot commented Oct 25, 2025 •
edited
Loading

✅ Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 64541d6
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69033dd7417aec0008045773
😎 Deploy Preview https://deploy-preview-52944--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 project configuration.

@developowl
Copy link
Member

/assign

ianychoi
ianychoi suggested changes Oct 27, 2025
View reviewed changes
Copy link
Member

@ianychoi ianychoi left a comment

Choose a reason for hiding this comment

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

The change part looks good me, but we need to double-check existing translation part to be consistent. Please double-check with my comments for your new revision. Thanks.

content/ko/docs/concepts/workloads/controllers/daemonset.md

{{< feature-state for_k8s_version="1.17" state="stable" >}}

데몬셋은 자격이 되는 모든 노드에서 파드 사본이 실행하도록 보장한다. 일반적으로
Copy link
Member

Choose a reason for hiding this comment

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

This line needs to be consistent with L21: "데몬셋 은 모든(또는 일부) 노드가 파드의 사본을 실행하도록 한다. 노드가 클러스터에 추가되면". I like "실행하도록 보장한다" - please reflect L21 translation.

content/ko/docs/concepts/workloads/controllers/daemonset.md Outdated
데몬셋은 파드를 생성한다는 점에서 [디플로이먼트](/ko/docs/concepts/workloads/controllers/deployment/)와 유사하고,
해당 파드에서는 프로세스가 종료되지 않을 것으로
예상한다(예: 웹 서버).
예상한다(예: 웹 서버, 스토리지 서버).
Copy link
Member

Choose a reason for hiding this comment

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

L236: "데몬 셋을" -> "데몬셋을"

developowl
developowl reviewed Oct 28, 2025
View reviewed changes
Copy link
Member

@developowl developowl left a comment

Choose a reason for hiding this comment

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

@Eundms 기여에 감사합니다. 간단한 리뷰를 남겼습니다. 확인 부탁드립니다!

content/ko/docs/concepts/workloads/controllers/daemonset.md Outdated
다음과 같이 관련 기능에 따라 자동적으로 데몬셋 파드에
톨러레이션을 추가한다.
데몬셋 컨트롤러는 데몬셋 파드에
{{< glossary_tooltip text="톨러레이션" term_id="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
{{< glossary_tooltip text="톨러레이션" term_id="toleration" >}} 세트를 추가한다.
{{< glossary_tooltip text="톨러레이션" term_id="toleration" >}} 세트를 자동으로 추가한다.

원문의 automatically 의 번역이 누락된 것 같습니다.

@lmktfy
Copy link
Member

lmktfy commented Oct 30, 2025

(thanks for the effort on updating this page)

@Eundms Eundms force-pushed the ko/concepts-workloads-controllers-daemonset branch from 274040c to 64541d6 Compare October 30, 2025 10:28
@Eundms
Copy link
Member Author

Eundms commented Oct 30, 2025

The change part looks good me, but we need to double-check existing translation part to be consistent. Please double-check with my comments for your new revision. Thanks.

@Eundms 기여에 감사합니다. 간단한 리뷰를 남겼습니다. 확인 부탁드립니다!

@ianychoi @developowl Thank you for the review! I’ve reflected your feedback and additionally fixed a few untranslated sentences and adjusted some wording in the table.

  • L161 Add translation데몬셋 파드는 상태가 좋지 않거나 파드를 수용할 준비가 되지 않은 노드에 스케줄링 될 수 있다.
  • L165 압력 -> 부족
  • L194 Add translation 목록
  • L224 몇 가지 이점 있다. -> 몇가지 장점이 있다

@ianychoi
Copy link
Member

/assign

/lgtm

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

LGTM label has been added.

Git tree hash: c33f17dfb25428603d880c2ac0c1a0aad6469b69

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seokho-son seokho-son Awaiting requested review from seokho-son

2 more reviewers

@developowl developowl developowl left review comments

@ianychoi ianychoi ianychoi requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ianychoi ianychoi

@developowl developowl

Labels

area/localization General issues or PRs related to localization 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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[ko] Update /docs/concepts/workloads/controllers/daemonset.md

5 participants

@Eundms @k8s-ci-robot @developowl @lmktfy @ianychoi