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

Update Zookeeper to 3.5.6 #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

vmakoto
Copy link

@vmakoto vmakoto commented Oct 26, 2019

Hello there, thanks for putting this repo together.

This patch updates the Dockerfile to use the latest version of zookeeper as of Oct 23rd.

It was tested against Kubernetes 1.14.8 managed by kops in AWS. The persistent volume used was EBS with xfs disk format

Special thanks to @c0rrzin.

Commits

remove hardcoded gpg verification

Add zookeeper authors keys to the repo and use it to verify the zookeeper artifacts

update dockerfile to use zookeeper 3.5.6

This patch updates the Dockerfile to pull zookeeper 3.5.6
from the right repo.

Also, since version 3.5.* version, most
of the 4 letter word commands must be whitelisted before they
can actually be used. This patch whitelists all of the during
zookeeper boot

@Slach
Copy link

Slach commented Jan 15, 2020

dear @kow3ns could you merge this pull request and upload new docker image to k8s.gcr.io ?

This patch updates the Dockerfile to pull zookeeper 3.5.6
from the right repo.

Also, since version 3.5.* version, most
of the 4 letter word commands must be whitelist before they
can actually be used. This patch whitelist all of the during
zookeeper boot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants