Re: code freeze for 4.5.0

2017-07-31 Thread Sijie Guo
On Mon, Jul 31, 2017 at 11:53 PM, Enrico Olivelli wrote: > Thank you Sijie > from my point of view the HttpServer is ready to go, will we include it in > 4.5 in your opinion ? > It is an isolated feature and off by default. I think it can go in 4.5 but we probably don't announce it in 4.5, but

Re: code freeze for 4.5.0

2017-07-31 Thread Enrico Olivelli
Thank you Sijie from my point of view the HttpServer is ready to go, will we include it in 4.5 in your opinion ? -- Enrico 2017-08-01 8:51 GMT+02:00 Sijie Guo : > On Mon, Jul 31, 2017 at 11:43 PM, Enrico Olivelli > wrote: > > > Great! > > > > some questions: > > - TLS support ? will this be i

[GitHub] sijie commented on issue #266: Issue 265: Add persistable bookie status

2017-07-31 Thread git
sijie commented on issue #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#issuecomment-319284324 @yzang I would probably defer this to 4.6.0. Is that okay to you? This is an au

Re: code freeze for 4.5.0

2017-07-31 Thread Sijie Guo
On Mon, Jul 31, 2017 at 11:43 PM, Enrico Olivelli wrote: > Great! > > some questions: > - TLS support ? will this be in ? > Yes. TLS will be in. > - There are a lot of flaky or constantly failing tests, shall we fix all of > them before preparing switching the master to 4.6 ? > Yes. We need t

Re: code freeze for 4.5.0

2017-07-31 Thread Enrico Olivelli
Great! some questions: - TLS support ? will this be in ? - There are a lot of flaky or constantly failing tests, shall we fix all of them before preparing switching the master to 4.6 ? I hope that most of us are testing current 4.5 snapshots against their applications in order to find regression

[jira] [Resolved] (BOOKKEEPER-1102) org.apache.bookkeeper.client.BookKeeperDiskSpaceWeightedLedgerPlacementTest.testDiskSpaceWeightedBookieSelectionWithBookiesBeingAdded is unreliable

2017-07-31 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-1102. --- Resolution: Fixed Fix Version/s: 4.5.0 Issue resolved by merging pull request 275

code freeze for 4.5.0

2017-07-31 Thread Sijie Guo
We've tried to clean up most of the issues marked for 4.5.0. I'd like to call a code freeze for 4.5.0. So no more issues/jiras should be marked in 4.5.0, unless it is really needed (e.g. bug fixes). Any new features should be marked as 4.6.0. I will work with JV to prepare the 4.5 branch for relea

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130529456 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java

[GitHub] sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130529148 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java ###

[GitHub] sijie closed pull request #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers

2017-07-31 Thread git
sijie closed pull request #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers URL: https://github.com/apache/bookkeeper/pull/275 This is an automated message from the Apache Git Service. To resp

[GitHub] eolivelli commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#issuecomment-319280896 @sijie for the "security". Yes I think it is not required and this time as httpserver will be turned off by default --

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130528376 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/http/ConfigurationService.java

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130528151 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java

[GitHub] sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130527487 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/http/ConfigurationService.java ##

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130527275 ## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/service/Service

[GitHub] sijie commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
sijie commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#issuecomment-319279024 @eolivelli the http port can be turned off by default. I don't see a strong point to include security at this point. we can come

[GitHub] sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130526935 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java ###

[GitHub] sijie closed issue #314: Enable checkstyle on bookeeper-server

2017-07-31 Thread git
sijie closed issue #314: Enable checkstyle on bookeeper-server URL: https://github.com/apache/bookkeeper/issues/314 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub an

[GitHub] sijie closed pull request #315: Issue 314: Enable checkstyle on bookkeeper-server

2017-07-31 Thread git
sijie closed pull request #315: Issue 314: Enable checkstyle on bookkeeper-server URL: https://github.com/apache/bookkeeper/pull/315 This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130526420 ## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/service/ServiceResp

[GitHub] sijie commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper

2017-07-31 Thread git
sijie commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper URL: https://github.com/apache/bookkeeper/pull/183#issuecomment-319277666 +1 @kishorekasi it seems that the latest push isn't compilable. can you fix that? we are ready to merge this one.

[GitHub] jiazhai opened a new issue #337: Docker image: K8s for bookkeeper and instructions on usage

2017-07-31 Thread git
jiazhai opened a new issue #337: Docker image: K8s for bookkeeper and instructions on usage URL: https://github.com/apache/bookkeeper/issues/337 **FEATURE REQUEST** 1. Please describe the feature you are requesting. K8s for bookkeeper and instructions on usage 2. In

[GitHub] jiazhai opened a new issue #336: Docker image: DCOS package update and instructions

2017-07-31 Thread git
jiazhai opened a new issue #336: Docker image: DCOS package update and instructions URL: https://github.com/apache/bookkeeper/issues/336 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Update DCOS package by using latest docker image; Update instructions

[GitHub] jiazhai opened a new issue #335: Docker Image: tracking sub issues

2017-07-31 Thread git
jiazhai opened a new issue #335: Docker Image: tracking sub issues URL: https://github.com/apache/bookkeeper/issues/335 Thanks to @sijie's summary, Here is the requirement discussed out in community meeting. - follow what other apache projects are doing [zeppelin](https://issues.apa

[GitHub] sijie opened a new pull request #334: Issue 321: Update "who are we?"

2017-07-31 Thread git
sijie opened a new pull request #334: Issue 321: Update "who are we?" URL: https://github.com/apache/bookkeeper/pull/334 Descriptions of the changes in this PR: - add JV to the PMC list - add siddharth and charan to the committers list - sort the people by apache ids -

Re: bookkeeper board report

2017-07-31 Thread Jia Zhai
Thanks for the instructions. On Tue, Aug 1, 2017 at 10:55 AM, Sijie Guo wrote: > Great! Thank you Jia. > > http://mail-archives.apache.org/mod_mbox/bookkeeper-dev/ > 201705.mbox/%3CCAO2yDyb4E508t-CDyqmqMB6mZ1a%2B3D_ > k6QKcchNOqMGqEwBe-g%40mail.gmail.com%3E > > This was the last board report dra

Re: bookkeeper board report

2017-07-31 Thread Sijie Guo
Great! Thank you Jia. http://mail-archives.apache.org/mod_mbox/bookkeeper-dev/201705.mbox/%3CCAO2yDyb4E508t-CDyqmqMB6mZ1a%2B3D_k6QKcchNOqMGqEwBe-g%40mail.gmail.com%3E This was the last board report draft. It should be a good example to follow. I can help you to grab the numbers. - Sijie On Mon,

[GitHub] sijie opened a new issue #333: [website] add bookkeeper slack channel to README.md and website

2017-07-31 Thread git
sijie opened a new issue #333: [website] add bookkeeper slack channel to README.md and website URL: https://github.com/apache/bookkeeper/issues/333 slack channel is live now. let's add it to README.md and also website. This

Slack Channel

2017-07-31 Thread Sijie Guo
Hi all, The slack channel https://apachebookkeeper.slack.com is live now. If you have emails (@apache.org), you can signup with https://apachebookkeeper.slack.com/signup. If you don't have apache emails, you can auto-invite yourself in https://apachebookkeeper.herokuapp.com/ If you were already

[GitHub] jiazhai commented on issue #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers

2017-07-31 Thread git
jiazhai commented on issue #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers URL: https://github.com/apache/bookkeeper/pull/275#issuecomment-319251425 Seems there is a conflicting file. This is

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

2017-07-31 Thread git
jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-319250335 @sijie, I am OK with your idea to start a new one, since this will be changed a lot. It depends on @caiok's decision. --

Re: bookkeeper board report

2017-07-31 Thread Jia Zhai
I would like to have a try. :) On Tue, Aug 1, 2017 at 7:57 AM, Sijie Guo wrote: > Hi all, > > We have a board report due in August. We have a pretty good progress in the > last 3 months. Anyone is interested in drafting a board report for August? > > - Sijie >

Re: Build failed in Jenkins: bookkeeper-postcommit-publish-website #15

2017-07-31 Thread Sijie Guo
INFRA team helped fix the problem. The CI job is running correctly now. Any changes made to website will be pushed automatically to bookkeeper.apache.org/test/content once they are merged. - Sijie On Mon, Jul 31, 2017 at 2:53 PM, Sijie Guo wrote: > The CI job can't get the credentials for pushi

bookkeeper board report

2017-07-31 Thread Sijie Guo
Hi all, We have a board report due in August. We have a pretty good progress in the last 3 months. Anyone is interested in drafting a board report for August? - Sijie

[GitHub] athanatos commented on issue #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers

2017-07-31 Thread git
athanatos commented on issue #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers URL: https://github.com/apache/bookkeeper/pull/275#issuecomment-319228009 @sijie @eolivelli Ok, updated. This is a

Re: [VOTE] Slack Channel for BookKeeper

2017-07-31 Thread Sijie Guo
Closed the vote. 4 binding +1 votes, 5 non-binding +1 votes. The vote is passed. I will proceed with the next steps to hand over the channel to BK PMC. Binding votes: - Ivan Kelly - JV - Matteo Merli - Sijie Guo Non binding votes: - atracymartin - Enrico Olivelli - Francesco Caliumi - Jia Zhai

Re: Build failed in Jenkins: bookkeeper-postcommit-publish-website #15

2017-07-31 Thread Sijie Guo
The CI job can't get the credentials for pushing the content to git. Need to talk to INFRA guys. - Sijie On Mon, Jul 31, 2017 at 2:47 PM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See publish-website/15/display/redirect?pa

Build failed in Jenkins: bookkeeper-postcommit-publish-website #15

2017-07-31 Thread Apache Jenkins Server
See Changes: [sijie] ISSUE #323: [WEBSITE] ci job to automatically publish website to apache -- [...truncated 167.28 KB...] delete mode 100644 content

[GitHub] sijie closed pull request #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie closed pull request #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/pull/332 This is an automated message from the

[GitHub] sijie closed issue #323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie closed issue #323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/issues/323 This is an automated message from the Apache Git Serv

[GitHub] sijie commented on issue #323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie commented on issue #323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/issues/323#issuecomment-319204888 @eolivelli I added the scripts at #332 and created https://builds.apache.org/view/BK/job/bo

Re: CI

2017-07-31 Thread Sijie Guo
Also I am not good at jenkins job builder. So if any people is good at jenkins job builder and willing to help the community to convert those jobs to use jenkins job builder, that would be great. - Sijie On Mon, Jul 31, 2017 at 2:37 PM, Sijie Guo wrote: > Hi all, > > I added one CI job 'bookkee

CI

2017-07-31 Thread Sijie Guo
Hi all, I added one CI job 'bookkeeper-postcommit-publish-website' to automatically generate and publish the content to bookkeeper.apache.org/test/content when changes are merged into master. The job is created here: - https://builds.apache.org/view/BK/job/bookkeeper-postcommit-publish-website/c

[GitHub] sijie commented on issue #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie commented on issue #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/pull/332#issuecomment-319200768 FYI The CI script: export MAVEN_HOME=/home/jenkins/tools/maven/la

[GitHub] sijie commented on issue #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie commented on issue #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/pull/332#issuecomment-319200768 FYI The CI script: ` export MAVEN_HOME=/home/jenkins/tools/mav

[GitHub] sijie commented on issue #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie commented on issue #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/pull/332#issuecomment-319200768 FYI The CI script: ` export MAVEN_HOME=/home/jenkins/tools/maven/l

[GitHub] sijie opened a new pull request #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged

2017-07-31 Thread git
sijie opened a new pull request #332: Issue 323: [website] ci job to automatically publish website to apache once the pull request is merged URL: https://github.com/apache/bookkeeper/pull/332 Descriptions of the changes in this PR: - add a Dockerfile to use ruby:2.4.1 to build websit

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

2017-07-31 Thread git
sijie commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-319171603 @jiazhai @caiok what is going to be reused here? as we discussed, we need to keep only one version. I am wondering why not just c

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130436791 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java ###

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130436818 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/http/JsonUtil.java ## @@

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130436614 ## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/service/ServiceResp

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130432413 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/http/ConfigurationService.java ##

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130435127 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java ###

[GitHub] athanatos commented on issue #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers

2017-07-31 Thread git
athanatos commented on issue #275: BOOKKEEPER-1102: Clarify BookieInfoReader and fix associated test flappers URL: https://github.com/apache/bookkeeper/pull/275#issuecomment-319126186 @eolivelli Will do today. This is an aut

Build failed in Jenkins: bookkeeper-master #1834

2017-07-31 Thread Apache Jenkins Server
See -- [...truncated 44.51 KB...] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.726 sec - in org.apache.bookkeeper.client.TestSequenceRead Running org.apache.bookkee

[GitHub] eolivelli opened a new issue #331: BookKeeper Benchmarks suite cannot run on non-English machines

2017-07-31 Thread git
eolivelli opened a new issue #331: BookKeeper Benchmarks suite cannot run on non-English machines URL: https://github.com/apache/bookkeeper/issues/331 **BUG REPORT** 1. Please describe the issue you observed: - What did you do? I was trying to run bookkeeper benchmarks usin

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130333140 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/http/ConfigurationService.java

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130332896 ## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/service/Service

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130332256 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

2017-07-31 Thread git
eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r130331025 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java

[GitHub] caiok commented on issue #328: Docker image: add a Docker Compose example file

2017-07-31 Thread git
caiok commented on issue #328: Docker image: add a Docker Compose example file URL: https://github.com/apache/bookkeeper/issues/328#issuecomment-319010704 Should this docker compose example start the dice tutorial application as currently does the Makefile? ---

[GitHub] caiok opened a new issue #330: Docker Image v1.0

2017-07-31 Thread git
caiok opened a new issue #330: Docker Image v1.0 URL: https://github.com/apache/bookkeeper/issues/330 This issue is for track the steps involved in order to implement the version 1.0 of the official bookkeeper docker image. I tried to enlist them in order of priority: - [ ] #197: fi

[GitHub] caiok opened a new issue #329: Docker image: gracefully handle start and stop bookie gracefully

2017-07-31 Thread git
caiok opened a new issue #329: Docker image: gracefully handle start and stop bookie gracefully URL: https://github.com/apache/bookkeeper/issues/329 We should modify run.sh to do a deleteCookie before bookie start and an autorecovery after its end, like done in @kishorekasi code: `C

[GitHub] caiok commented on issue #288: Docker image: provide a template related way to handle different version

2017-07-31 Thread git
caiok commented on issue #288: Docker image: provide a template related way to handle different version URL: https://github.com/apache/bookkeeper/issues/288#issuecomment-319007210 @jiazhai Should this issue be dropped after the community meeting decisions or it's still needed? --

[GitHub] caiok commented on issue #289: Docker image: make docker image build based on master latest code

2017-07-31 Thread git
caiok commented on issue #289: Docker image: make docker image build based on master latest code URL: https://github.com/apache/bookkeeper/issues/289#issuecomment-319006195 @jiazhai This issue is meant to creating an additional build or replace the existing one? I tend for the former. -

[GitHub] caiok commented on issue #328: Docker image: add a Docker Compose example file

2017-07-31 Thread git
caiok commented on issue #328: Docker image: add a Docker Compose example file URL: https://github.com/apache/bookkeeper/issues/328#issuecomment-318991273 This issue it's a good opportunity to decide whether the example in Makefile should be kept or it's better to drop it. And if we choo

[GitHub] caiok opened a new issue #328: Docker image: add a Docker Compose example file

2017-07-31 Thread git
caiok opened a new issue #328: Docker image: add a Docker Compose example file URL: https://github.com/apache/bookkeeper/issues/328 Now that we addressed the issue of first ensemble start adding an optional metaformat before running the bk server, we should add a docker compose example for

[GitHub] caiok opened a new issue #327: Docker image: Drop versions and Alpine support

2017-07-31 Thread git
caiok opened a new issue #327: Docker image: Drop versions and Alpine support URL: https://github.com/apache/bookkeeper/issues/327 We need to drop alpine support and multiple versions support and move all docker related files in docker main directory.

[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