[GitHub] [bookkeeper] eolivelli opened a new pull request #3149: BookieAutoRecoveryTest.testEmptyLedgerLosesQuorumEventually fix flaky test, ensure that the Auditor is alive

[GitBox <gi...@apache.org>]

eolivelli opened a new pull request #3149:
URL: https://github.com/apache/bookkeeper/pull/3149


   ### Motivation
   The test testEmptyLedgerLosesQuorumEventually kills two bookies, and very often it kills the Auditor Bookie.
   
   ### Changes
   - BookKeeperClusterTestCase.getAuditor(), ensure that we pick up a Auditor that is running and that is not going to die soon
   - ReplicationWorker exist the main loop in case of InterruptedException masked by a UnavailableException (this reduces a lot the spam in the logs, because in any case the thread is deemed to always fail for InterruptedExceptions)
   - in testEmptyLedgerLosesQuorumEventually ensure that the Auditor runs and start a new Bookie before the final check (the Auditor does nothing if the is no "bookie to audit", so there are races in which if the cluster does not change the Auditor does not kick in in any case)


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [bookkeeper] eolivelli merged pull request #3149: BookieAutoRecoveryTest.testEmptyLedgerLosesQuorumEventually fix flaky test, ensure that the Auditor is alive

[GitBox <gi...@apache.org>]

eolivelli merged pull request #3149:
URL: https://github.com/apache/bookkeeper/pull/3149


   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org