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: Add tutorials/stateful-application/zookeeper #12452 #13523

Merged
merged 3 commits into from Apr 23, 2019

Conversation

yoonian
Copy link
Contributor

@yoonian yoonian commented Mar 29, 2019

No description provided.

@k8s-ci-robot k8s-ci-robot added 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/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Mar 29, 2019
@netlify
Copy link

netlify bot commented Mar 29, 2019

Deploy preview for k8s-dev-ko ready!

Built with commit 535c970

https://deploy-preview-13523--k8s-dev-ko.netlify.com

@claudiajkang
Copy link
Contributor

/assign

Copy link
Contributor

@claudiajkang claudiajkang left a comment

Choose a reason for hiding this comment

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

@yoonian PR 감사합니다.

리뷰 늦어진 점 사과드립니다. 양이 방대하여 일단은 453 라인까지 리뷰 완료했습니다. 그 이후 라인은 빠른 시일이내 리뷰 하도록 하겠습니다.
리뷰 내용은 크게 다음과 같고, 어색한 부분의 경우 임의로 수정하였습니다. 그 부분은 선택적으로 반영해서 적용해주시면 됩니다.

리뷰내용

  • 리뷰어 삭제
  • 상용 프로그램명의 경우 원문 유지 : 주키퍼 -> Zookeeper
  • 원문과 비교하여 어색한 내용 수정
  • 너무 긴 문장의 경우 , 추가
  • 원문과 비교하여 존재하지 않는 부분 삭제
  • 오타 수정
  • 명령형 문장 청유형으로 변경(-하라 -> 한다)

@claudiajkang claudiajkang force-pushed the dev-1.14-ko.1 branch 3 times, most recently from 42e470c to 9de4a6e Compare April 9, 2019 16:39
@gochist
Copy link
Member

gochist commented Apr 11, 2019

@yoonian 4월 9일에 dev-1.14-ko.1master에 머지되었습니다. 베이스 브랜치를 dev-1.14-ko.2로 변경 바랍니다. dev-1.14-ko.2 릴리스 스케줄은 한글화팀 회의록에서 확인하실 수 있습니다.
/hold

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 11, 2019
@claudiajkang claudiajkang changed the base branch from dev-1.14-ko.1 to dev-1.14-ko.2 April 13, 2019 15:47
Copy link
Contributor

@claudiajkang claudiajkang left a comment

Choose a reason for hiding this comment

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

507라인까지 추가리뷰입니다.


## 일관된 구성 보장하기

As noted in the [Facilitating Leader Election](#facilitating-leader-election) and
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
As noted in the [Facilitating Leader Election](#facilitating-leader-election) and
  • 원문 삭제

## 일관된 구성 보장하기

As noted in the [Facilitating Leader Election](#facilitating-leader-election) and
[Achieving Consensus](#achieving-consensus) sections, the servers in a
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[Achieving Consensus](#achieving-consensus) sections, the servers in a


As noted in the [Facilitating Leader Election](#facilitating-leader-election) and
[Achieving Consensus](#achieving-consensus) sections, the servers in a
ZooKeeper ensemble require consistent configuration to elect a leader
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ZooKeeper ensemble require consistent configuration to elect a leader

As noted in the [Facilitating Leader Election](#facilitating-leader-election) and
[Achieving Consensus](#achieving-consensus) sections, the servers in a
ZooKeeper ensemble require consistent configuration to elect a leader
and form a quorum. They also require consistent configuration of the Zab protocol
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
and form a quorum. They also require consistent configuration of the Zab protocol

[Achieving Consensus](#achieving-consensus) sections, the servers in a
ZooKeeper ensemble require consistent configuration to elect a leader
and form a quorum. They also require consistent configuration of the Zab protocol
in order for the protocol to work correctly over a network. In our example we
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
in order for the protocol to work correctly over a network. In our example we


[리더 선출 촉진](#facilitating-leader-election)과
[합의 달성](#achieving-consensus) 섹션에서 알렸듯이,
주키퍼 앙상블에 서버는 리더 선출과 쿼럼을 구성하기 위한 일관된 설정이 필요하다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
주키퍼 앙상블에 서버는 리더 선출과 쿼럼을 구성하기 위한 일관된 설정이 필요하다.
ZooKeeper 앙상블의 서버는 리더 선출과 쿼럼을 구성하기 위한 일관된 설정이 필요하다.

[리더 선출 촉진](#facilitating-leader-election)과
[합의 달성](#achieving-consensus) 섹션에서 알렸듯이,
주키퍼 앙상블에 서버는 리더 선출과 쿼럼을 구성하기 위한 일관된 설정이 필요하다.
또한 Zab 프로토콜의 일관된 설정도 네트워크에 걸쳐 올바르게 동작하기 위해서
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
또한 Zab 프로토콜의 일관된 설정도 네트워크에 걸쳐 올바르게 동작하기 위해서
또한 Zab 프로토콜의 일관된 설정도
네트워크에 걸쳐 올바르게 동작하기 위해서

```

주키퍼 서버를 시작하는데 사용한 명령어는 커맨드라인 파라미터로 환경 구성을 전달했다. 환경 변수를 이용하여서도 앙상블에 환경 구성을 전달할 수 있다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
주키퍼 서버를 시작하는데 사용한 명령어는 커맨드라인 파라미터로 환경 구성을 전달했다. 환경 변수를 이용하여서도 앙상블에 환경 구성을 전달할 수 있다.
ZooKeeper 서버를 시작하는데 사용한 명령어는 커맨드라인 파라미터로 환경 구성을 전달했다. 환경 변수를 이용하여서도 앙상블에 환경 구성을 전달할 수 있다.


### 로깅 설정하기

`zkGenConfig.sh` 스크립트로 생성된 파일 중 하나는 주키퍼의 로깅을 제어한다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
`zkGenConfig.sh` 스크립트로 생성된 파일 중 하나는 주키퍼의 로깅을 제어한다.
`zkGenConfig.sh` 스크립트로 생성된 파일 중 하나는 ZooKeeper의 로깅을 제어한다.

### 로깅 설정하기

`zkGenConfig.sh` 스크립트로 생성된 파일 중 하나는 주키퍼의 로깅을 제어한다.
주키퍼는 [Log4j](http://logging.apache.org/log4j/2.x/)를 이용하며
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
주키퍼는 [Log4j](http://logging.apache.org/log4j/2.x/)를 이용하며
ZooKeeper는 [Log4j](http://logging.apache.org/log4j/2.x/)를 이용하며

Copy link
Contributor

@claudiajkang claudiajkang left a comment

Choose a reason for hiding this comment

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

@yoonian 파일 전체 리뷰 완료했습니다~ 확인 부탁드립니다.

git pull 작업을 먼저 진행하신 이후에 리뷰 내용 반영 부탁드립니다.

kubectl exec zk-0 cat /usr/etc/zookeeper/log4j.properties
```

아래 로깅 구성은 ZooKeeper가 모든 로그를 표준 출력 스트림으로 처리하게 한다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
아래 로깅 구성은 ZooKeeper가 모든 로그를 표준 출력 스트림으로 처리하게 한다.
아래 로깅 구성은 ZooKeeper가 모든 로그를
표준 출력 스트림으로 처리하게 한다.

log4j.appender.CONSOLE.layout.ConversionPattern=%d{ISO8601} [myid:%X{myid}] - %-5p [%t:%C{1}@%L] - %m%n
```

이는 컨테이너 내에서 안전하게 로깅하는 가장 단순한 방법이다. 애플리케이션이 표준 출력으로 로그를 쓰면 쿠버네티스는 로그 로테이션을 처리한다. 또한 쿠버네티스는 애플리케이션이 표준 출력과 표준 오류에 쓰여진 로그로 인하여 로컬 저장 미디어가 고갈되지 않도록 보장하는 정상적인 보존 정책을 구현한다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
이는 컨테이너 내에서 안전하게 로깅하는 가장 단순한 방법이다. 애플리케이션이 표준 출력으로 로그를 쓰면 쿠버네티스는 로그 로테이션을 처리한다. 또한 쿠버네티스는 애플리케이션이 표준 출력과 표준 오류에 쓰여진 로그로 인하여 로컬 저장 미디어가 고갈되지 않도록 보장하는 정상적인 보존 정책을 구현한다.
이는 컨테이너 내에서 안전하게 로깅하는 가장 단순한 방법이다. 표준 출력으로 애플리케이션 로그를 작성하면, 쿠버네티스는 로그 로테이션을 처리한다. 또한 쿠버네티스는 애플리케이션이 표준 출력과 표준 오류에 쓰여진 로그로 인하여 로컬 저장 미디어가 고갈되지 않도록 보장하는 정상적인 보존 정책을 구현한다.

@claudiajkang
Copy link
Contributor

/lgtm
/approve

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

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ClaudiaJKang

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 Apr 23, 2019
@claudiajkang
Copy link
Contributor

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 23, 2019
@k8s-ci-robot k8s-ci-robot merged commit 7661b33 into kubernetes:dev-1.14-ko.2 Apr 23, 2019
claudiajkang pushed a commit that referenced this pull request Apr 23, 2019
* Update outdated files in dev-1.14-ko.2 partly (#13879)

* Update outdated files in dev-1.14-ko.2 partly-concepts-contribute-ref (#13890)

* ko: update dev-1.14-ko.2 for docs/tutorials/configuration/configure-redis-using-configmap.md #13738 (#13891)

* Ko trans: translate contribute/participating.md in Korean (#13812)

* Update outdated files of setup and tasks in dev-1.14-ko.2 partly (#13893)

* ko-trans: Fix typo in install-minikube (#13894)

* ko: Add tutorials/stateful-application/zookeeper #12452 (#13523)

* ko: Add tutorials/services/source-ip #12457 (#13524)

Co-authored-by:    Yoon <learder@gmail.com>

Co-authored-by:    June Yi <june.yi@samsung.com>

Co-authored-by:    Seokho <shsongist@gmail.com>
claudiajkang pushed a commit that referenced this pull request Apr 23, 2019
* Update outdated files in dev-1.14-ko.2 partly (#13879)

* Update outdated files in dev-1.14-ko.2 partly-concepts-contribute-ref (#13890)

* ko: update dev-1.14-ko.2 for docs/tutorials/configuration/configure-redis-using-configmap.md #13738 (#13891)

* Ko trans: translate contribute/participating.md in Korean (#13812)

* Update outdated files of setup and tasks in dev-1.14-ko.2 partly (#13893)

* ko-trans: Fix typo in install-minikube (#13894)

* ko: Add tutorials/stateful-application/zookeeper #12452 (#13523)

* ko: Add tutorials/services/source-ip #12457 (#13524)

Co-authored-by:    Yoon <learder@gmail.com>
Co-authored-by:    June Yi <june.yi@samsung.com>
Co-authored-by:    Seokho <shsongist@gmail.com>
k8s-ci-robot pushed a commit that referenced this pull request Apr 23, 2019
* Update outdated files in dev-1.14-ko.2 partly (#13879)

* Update outdated files in dev-1.14-ko.2 partly-concepts-contribute-ref (#13890)

* ko: update dev-1.14-ko.2 for docs/tutorials/configuration/configure-redis-using-configmap.md #13738 (#13891)

* Ko trans: translate contribute/participating.md in Korean (#13812)

* Update outdated files of setup and tasks in dev-1.14-ko.2 partly (#13893)

* ko-trans: Fix typo in install-minikube (#13894)

* ko: Add tutorials/stateful-application/zookeeper #12452 (#13523)

* ko: Add tutorials/services/source-ip #12457 (#13524)

Co-authored-by:    Yoon <learder@gmail.com>
Co-authored-by:    June Yi <june.yi@samsung.com>
Co-authored-by:    Seokho <shsongist@gmail.com>
@yoonian yoonian deleted the dev-1.14-ko.1-12452 branch April 30, 2019 15:37
@yoonian yoonian restored the dev-1.14-ko.1-12452 branch April 30, 2019 15:37
@yoonian yoonian deleted the dev-1.14-ko.1-12452 branch April 30, 2019 15:42
nowjean pushed a commit to nowjean/website that referenced this pull request Jun 13, 2019
* Update outdated files in dev-1.14-ko.2 partly (kubernetes#13879)

* Update outdated files in dev-1.14-ko.2 partly-concepts-contribute-ref (kubernetes#13890)

* ko: update dev-1.14-ko.2 for docs/tutorials/configuration/configure-redis-using-configmap.md kubernetes#13738 (kubernetes#13891)

* Ko trans: translate contribute/participating.md in Korean (kubernetes#13812)

* Update outdated files of setup and tasks in dev-1.14-ko.2 partly (kubernetes#13893)

* ko-trans: Fix typo in install-minikube (kubernetes#13894)

* ko: Add tutorials/stateful-application/zookeeper kubernetes#12452 (kubernetes#13523)

* ko: Add tutorials/services/source-ip kubernetes#12457 (kubernetes#13524)

Co-authored-by:    Yoon <learder@gmail.com>
Co-authored-by:    June Yi <june.yi@samsung.com>
Co-authored-by:    Seokho <shsongist@gmail.com>
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/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants