Translate tasks/tls/certificate-rotation/ in Korean #21838
Conversation
k8s-ci-robot
added
cncf-cla: yes
k8s-ci-robot
added
language/ko
|
Deploy preview for k8s-dev-ko ready! Built with commit a634711 |
|
/assign @seokho-son |
There was a problem hiding this comment.
@marcushyungseoklee 컨트리뷰션 감사합니다 !! ^^
리뷰를 일부 진행하다가 보니,
한글화 작업을 dev-1.18-ko.6 의 영문 문서가 아니라
이전 브랜치의 영문 문서(outdated)를 기준으로 진행하신 것 같습니다.
내용에 대한 리뷰 이전에
해당 사항(dev-1.18-ko.6 의 영문 문서를 기준으로 한글화)을 다시 확인하셔서 PR 업데이트 해주시길 부탁드립니다. 이후에 내용에 대한 리뷰를 진행하겠습니다. 감사합니다.. ^^
|
dev-1.18-ko.6 기준으로 재 확인해서 다시 PR 업데이트 하겠습니다 |
|
@seokho-son 말씀주신대로 1.18-ko6 en의 파일을 확인하여 ko버전의 certificate-rotation.md 을 다시 push 하였습니다. 감사합니다. |
There was a problem hiding this comment.
@marcushyungseoklee
업데이트 감사합니다. 내용에 대한 리뷰 의견 남겼습니다.
참고하셔서 업데이트 부탁 드립니다 ^^
|
@seokho-son 리뷰 감사합니다^^ |
|
@marcushyungseoklee 리뷰 의견 중에 몇 가지 누락된 사항이 있는 것 같습니다. 확인 부탁 드리며, |
| `--cert-dir`에 지정된 위치에 디스크에 기록한다. | ||
| 그런 다음 kubelet은 쿠버네티스 API에 연결해서 새로운 인증서를 사용한다. | ||
|
|
||
| 서명된 인증서가 만료가 다가오면 kubelet은 쿠버네티스 API를 사용하여 |
There was a problem hiding this comment.
| 서명된 인증서가 만료가 다가오면 kubelet은 쿠버네티스 API를 사용하여 | |
| 서명된 인증서의 만료가 다가오면 kubelet은 쿠버네티스 API를 사용하여 |
수정 사항을 확인해보니, 사소한 오류가 아직 남아 있는 것 같아서 의견 드립니다.
해당 사항에 대한 수정 후에 바로 Commit squashing을 부탁 드립니다..^^
Commit squashing에 어려움이 있다면 알려주시기 바랍니다... ^^
There was a problem hiding this comment.
commit squishing을 했다고 생각했는데 무언가 conflict가 생긴것 같습니다..
일단 리뷰 주신 부분 다시 확인해서 수정했습니다
Commit squashing을 첨 하다보니 좀 진행이 더디네요..
There was a problem hiding this comment.
확인해보니 squishing을 진행하고 나서 코드가 수정전의 코드로 되돌아갔네요 먼가 문제가 있었던것 같습니다. 일단 다시 코멘트 첨부터 확인해서 정상수정되었는지 확인하고 코멘트 달도록 하겠습니다
There was a problem hiding this comment.
확인해보니 squishing을 진행하고 나서 코드가 수정전의 코드로 되돌아갔네요 먼가 문제가 있었던것 같습니다. 일단 다시 코멘트 첨부터 확인해서 정상수정되었는지 확인하고 코멘트 달도록 하겠습니다
저도 가끔씩 코드가 날아가서, 별도의 파일에 수정하고 website/ 아래로 복사하는 방법을 쓰고 있습니다. ^^
There was a problem hiding this comment.
ㅠㅠ 슬프네요..금일 다시 반영예정입니다
There was a problem hiding this comment.
@marcushyungseoklee 업데이트 감사합니다. 고생 많으셨어요.
사소한 사항이 누락된 것 같아서 아래와 같이 의견 드립니다.
마찬가지로 commits squashing을 부탁드립니다.
그리고 혹시 이번에 commits squashing 작업이 까다로우시면, 알려주세요.
한글 작업 브랜치에서는 commits squashing이 의무 사항이 아니기 때문에 2~3개 정도의 commits 는 수용이 가능합니다.
(master 브랜치로의 PR에는 commit squashing이 의무입니다. 참고하시기 바랍니다.)
apply all comments apply two additional comments
|
@seokho-son 마지막 말씀 주신 부분까지 수정하여 업데이트 하였습니다 :) |
|
감사합니다 ! |
k8s-ci-robot
added
the
lgtm
|
[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 |
k8s-ci-robot
added
the
approved
- Update outdated files in dev-1.18-ko.6 partly (#21714) - Translate StorageClass to Korean word (#21805) - Translate StatefulSet to Korean word in pods.md (#21794) - Translate tasks/run-application/delete-stateful-set/ in Korean (#21686) - Translate tasks/administer-cluster/change-default-storage-class in Korean (#21801) - update outdated docs (#21940) - Modify spacing term StatefulSet in Korean (#21871) - Translate /tasks/administer-cluster/coredns in Korean (#21876) - Fix typo in k8s.io/ko/docs/contribute/new-content/new-content/ (#21975) - Translation error correction in /concepts/containers/runtime-class.md (#21868) - Translate tasks/tls/certificate-rotation/ in Korean (#21838) - Translate /tasks/configure-pod-container/static-pod in Korean (#21798) - Update to Outdated files in the dev-1.18-ko.6 branch - (1/4) (#21911) - Fix English bugs in Korean documentation (#21994) - Update outdated dev-1.18-ko.6 partly (#22067) Co-authored-by: Jerry Park <[email protected]> Co-authored-by: PyungHo Yoon <[email protected]> Co-authored-by: Yuk, Yongsu <[email protected]> Co-authored-by: Jerry Park <[email protected]> Co-authored-by: Seokho Son [email protected] Co-authored-by: Jordy Ruiter <[email protected]> Co-authored-by: bluefriday <[email protected]> Co-authored-by: Dajin Gwon <[email protected]> Co-authored-by: Hyungseok Lee <[email protected]> Co-authored-by: Jihoon Seo <[email protected]> Co-authored-by: coolguyhong <[email protected]> Co-authored-by: jmyung <[email protected]> Co-authored-by: June Yi <[email protected]>
- Update outdated files in dev-1.18-ko.6 partly (#21714) - Translate StorageClass to Korean word (#21805) - Translate StatefulSet to Korean word in pods.md (#21794) - Translate tasks/run-application/delete-stateful-set/ in Korean (#21686) - Translate tasks/administer-cluster/change-default-storage-class in Korean (#21801) - update outdated docs (#21940) - Modify spacing term StatefulSet in Korean (#21871) - Translate /tasks/administer-cluster/coredns in Korean (#21876) - Fix typo in k8s.io/ko/docs/contribute/new-content/new-content/ (#21975) - Translation error correction in /concepts/containers/runtime-class.md (#21868) - Translate tasks/tls/certificate-rotation/ in Korean (#21838) - Translate /tasks/configure-pod-container/static-pod in Korean (#21798) - Update to Outdated files in the dev-1.18-ko.6 branch - (1/4) (#21911) - Fix English bugs in Korean documentation (#21994) - Update outdated dev-1.18-ko.6 partly (#22067) Co-authored-by: Jerry Park <[email protected]> Co-authored-by: PyungHo Yoon <[email protected]> Co-authored-by: Yuk, Yongsu <[email protected]> Co-authored-by: Jerry Park <[email protected]> Co-authored-by: Seokho Son <[email protected]> Co-authored-by: Jordy Ruiter <[email protected]> Co-authored-by: bluefriday <[email protected]> Co-authored-by: Dajin Gwon <[email protected]> Co-authored-by: Hyungseok Lee <[email protected]> Co-authored-by: Jihoon Seo <[email protected]> Co-authored-by: coolguyhong <[email protected]> Co-authored-by: jmyung <[email protected]> Co-authored-by: June Yi <[email protected]>
from #20935