New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test(raft): remove unnecessary test #8479
Conversation
The test was flaky. It is not clear what property this test is verifying. The current one is inherentaly non-deterministic as it expects a specific leader nodes which is not possible to guarantee.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Should we backport this?
bors merge |
8479: test(raft): remove unnecessary test r=deepthidevaki a=deepthidevaki ## Description Removed a test which was flaky. It is not clear what property this test is verifying. The current one is inherentaly non-deterministic as it expects a specific leader nodes which is not possible to guarantee. ## Related issues closes #8417 Co-authored-by: Deepthi Devaki Akkoorath <deepthidevaki@gmail.com>
Build failed: |
bors retry Hm, weird, it ran out of memory. Last I had checked, we didn't even get close to the container's memory limits...but maybe instead we're busting the |
Successfully created backport PR #8489 for |
Successfully created backport PR #8490 for |
Description
Removed a test which was flaky. It is not clear what property this test is verifying. The current one is inherentaly non-deterministic as it expects a specific leader nodes which is not possible to guarantee.
Related issues
closes #8417
Definition of Done
Not all items need to be done depending on the issue and the pull request.
Code changes:
backport stable/0.25
) to the PR, in case that fails you need to create backports manually.Testing:
Documentation: