You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2020/04/12 22:46:36 UTC

[GitHub] [bookkeeper] Ghatage opened a new pull request #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.

Ghatage opened a new pull request #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.
URL: https://github.com/apache/bookkeeper/pull/2306
 
 
   ### Motivation
   Currently we allow bookies to come up even if they don't have a cookie in the metadata store.
   The cookie is an identity of a registered bookie and a bookie should not be allowed to come up without it.
   
   ### Changes
   Update handling of bookie boot up. Add test.
   This scenario is rarely expected but can happen in case of corruption or errant deletion of znodes for cookies
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [bookkeeper] Ghatage commented on issue #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.

Posted by GitBox <gi...@apache.org>.
Ghatage commented on issue #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.
URL: https://github.com/apache/bookkeeper/pull/2306#issuecomment-613069101
 
 
   Hi @eolivelli,
   Thanks for the review, I fixed the errors.
   Now it seems like a Github Actions error in the Integration tests.
   
   ```
   [ERROR] The command '/bin/sh -c /opt/bookkeeper/scripts/install-python-client.sh' returned a non-zero code: 1
   [ERROR] The command '/bin/sh -c /opt/bookkeeper/scripts/install-python-client.sh' returned a non-zero code: 1
   [ERROR] Failed to execute goal com.spotify:dockerfile-maven-plugin:1.4.13:build (default) on project current-version-image: Could not build image: The command '/bin/sh -c /opt/bookkeeper/scripts/install-python-client.sh' returned a non-zero code: 1 -> [Help 1]
   [ERROR] 
   [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
   [ERROR] Re-run Maven using the -X switch to enable full debug logging.
   [ERROR] 
   [ERROR] For more information about the errors and possible solutions, please read the following articles:
   [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
   [ERROR] 
   [ERROR] After correcting the problems, you can resume the build with the command
   [ERROR]   mvn <args> -rf :current-version-image
   ##[error]Process completed with exit code 1.
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [bookkeeper] Ghatage commented on issue #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.

Posted by GitBox <gi...@apache.org>.
Ghatage commented on issue #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.
URL: https://github.com/apache/bookkeeper/pull/2306#issuecomment-612687748
 
 
   Fix for #2305. 
   @eolivelli 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

[GitHub] [bookkeeper] eolivelli commented on issue #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.

Posted by GitBox <gi...@apache.org>.
eolivelli commented on issue #2306: Issue #2305: Bookies should not be allowed to come up without a cookie in metadata store.
URL: https://github.com/apache/bookkeeper/pull/2306#issuecomment-612768500
 
 
   Please check CI

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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