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 2022/01/07 14:14:08 UTC
[GitHub] [bookkeeper] nicoloboschi opened a new pull request #2978: Backward compatibility tests: add retries
nicoloboschi opened a new pull request #2978:
URL: https://github.com/apache/bookkeeper/pull/2978
### Motivation
Backward compatibility tests are flaky. When they fail, they always fail due to
```
Not enough non-faulty bookies available
```
I spent a lot of time trying to tune connection parameters, split tests and so on, my understanding is that at certain points the machine starts suffering (he restart many times the bookies of different versions - 3 bookies and 1 zk in the same machine) and the bookie is unable to send ZK ping and then it's cut off from the ensemble.
### Changes
I added the retry options to the tests with max 5 retries per test, it should be enough. (I don't have numbers to demonstrate that, we'll see it )
--
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] pkumar-singh merged pull request #2978: Backward compatibility tests: add retries
Posted by GitBox <gi...@apache.org>.
pkumar-singh merged pull request #2978:
URL: https://github.com/apache/bookkeeper/pull/2978
--
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] pkumar-singh merged pull request #2978: Backward compatibility tests: add retries
Posted by GitBox <gi...@apache.org>.
pkumar-singh merged pull request #2978:
URL: https://github.com/apache/bookkeeper/pull/2978
--
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