[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-08-01 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-319306295 @jiazhai As a first thing we should decide the issues we want included in this PR. I suggest #327 and the base step for #289 that

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-08-01 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-319291368 @sijie @jiazhai I agree, let's close this PR and start a new one. I will work on it as soon as possible. --

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-31 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-318988058 Thanks @sijie for the recap of community meeting decisions. @jiazhai I'm ok with merging this and move on. I will open new

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-25 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317797541 @sijie Thanks for the reply. We don't push images on dockerhub, dockerhub checkout our dockerfiles and builds them. Check [this

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-25 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317744941 @sijie @fpj @merlimat Any other doubts or requests for this PR? We need it merged before proceeding with the related issues (and

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-21 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317041008 @jiazhai What are your concerns about this code duplication? I think providing a centos image for who is concerned about lice

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-19 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-316403959 @jiazhai Seems that Apache examples are a bit poor on this side. [Here an example](https://hub.docker.com/r/azul/zulu-openjdk/) (

[GitHub] caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-19 Thread git
caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-316344903 @jiazhai It's ok to have such a structure. We need it in order to provide docker images of older versions. Having a CentOS im