[GitHub] ArvinDevel commented on issue #470: Issue #430: Add an instruction in docker README to run standalone cluster

[gi...@git.apache.org]

ArvinDevel commented on issue #470: Issue #430: Add an instruction in docker README to run standalone cluster
URL: https://github.com/apache/bookkeeper/pull/470#issuecomment-325129086
 
 
   During my test, I found somethings.
   First, the JAVA_HOME  env is not set in current Docker image, so every time I enter the container to execute the bookkeeper command, I should set it firstly, I think we can set it in entrypoint.sh or somewhere else.
   Second,When I use the Makefile to setup cluster in Mac, there exists some error:
   If I execute `make run-format`, the info "/opt/bookkeeper/entrypoint.sh: line 51: sudo: command not found" showing that the current image don't have 'sudo' command;
   if I execute `make run-bk` after execute `make run-zk`, the info "/opt/bookkeeper/entrypoint.sh: line 62: /opt/zk/bin/zkCli.sh: No such file or directory" showing the image lack 'zkCli.sh'; These two little matter doesn't make sense, we can improve it later.
   if I restart the zk by run `make run-zk` again after shutdown it, then the `make run-bk` will have the info" ERROR - [main:Bookie@391] - Couldn't access cookie in zookeeper
   org.apache.zookeeper.KeeperException$NodeExistsException: KeeperErrorCode = NodeExists for /ledgers/cookies/172.18.0.3:3181", I found this was due to the binding to local dir, so should we clear the local dir when we start zookeeper? 
   Third, current Makefile pull  bookkeeper/bookie:centos docker image from store, and we use apache/bookkeeper in the ReadMe file, should we keep consistence?
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services