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 website/docs/concepts/storage/windows-storage/ into Korean #35698

Conversation

NayeonKeum
Copy link
Contributor

Signed-off-by: Nayeon Keum [email protected]

issue: #35184

@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. language/ko Issues or PRs related to Korean language labels Aug 4, 2022
@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 seokho-son for approval by writing /assign @seokho-son in a comment. For more information see:The Kubernetes 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 the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Aug 4, 2022
@netlify
Copy link

netlify bot commented Aug 4, 2022

Deploy Preview for k8s-dev-ko ready!

Name Link
🔨 Latest commit 08c8234
🔍 Latest deploy log https://app.netlify.com/sites/k8s-dev-ko/deploys/62eb9b548e3f1f00088ce936
😎 Deploy Preview https://deploy-preview-35698--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

@NayeonKeum 님~ 기여 감사드립니다!! :)

올리신 이슈에 따라서, 작업을 잘 해주신 것 같습니다.

그런데, 추가된 문서와 동일한 문서가
이미 #35423 에서 작업이되고 있었습니다. ㅠㅠ

원래라면, @NayeonKeum 님이 #35184 이슈를 통해서 작업 예정이라고 알려주셨기 때문에
우선권을 가지고 계신다고 할 수 있습니다.

그런데, PR #35423 의 경우, 한글화 마일스톤 관리 차원에서 생성된 PR이라,
리뷰까지 추진이 되어버린 상태입니다. (거의 마무리 단계에 있습니다.)

리뷰어 및 승인자가 상황을 잘 파악했다면,
#35423 가 오픈되었거나 관련된 이슈가 생성되었을 때, 해당 정보를 @NayeonKeum 님께 미리 알려드렸어야 하는게 맞습니다.
이번에는 outdated 된 내용이 다소 복잡하게 되어 있어서, 이를 추적하지 못하였습니다.

참고: PR #35423 은 최근 업스트림에서
장문의 content/en/docs/setup/production-environment/windows/intro-windows-in-kubernetes.md 가
content/en/docs/concepts/storage/windows-storage.md 등 여러개의 별도 문서로 분할된 상황을
따라잡고자, 관련된 사항을 한꺼번에 처리하고 있습니다. (향후에도 유사한 상황이 발생할 수 있으니, PR 제목 등에 조금 더 신경을 써야겠네요.)


아무튼, 일반적인 상황이 아니라 기여자간 작업을 추적하지 못하고,
일부 중복 작업을 하시게 만들었네요 ㅠㅠ 양해를 부탁드립니다.

현 상황에서 저의 제안은 다음과 같습니다.

  • 현 PR은 잠시 hold 또는 close하고,
  • PR [ko] Reorg 'Windows in K8s' docs #35423 가 dev-1.24-ko.2 및 main까지 머지된 후,
  • dev.1.24-ko.3 (조만간 생성) 에서,
  • 현 PR의 기여 내용을 활용하여, ko/docs/concepts/storage/windows-storage.md 를
    개선하는 신규 PR로 기여 내용을 추가적으로 반영.

이렇게 처리하면 어떨까요?

(cc @jihoon-seo)

@jihoon-seo
Copy link
Member

@seokho-son 네 말씀하신 대로 하는 것이 좋을 것 같습니다. 😊

@NayeonKeum
Copy link
Contributor Author

넵 좋습니다! PR은 Close 하도록 하겠습니다. 신경 써주셔서 감사합니다 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/ko Issues or PRs related to Korean language 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