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
CASSANDRA-16411 CASSANDRA-16296 Fix topology corruption on joining no… #727
Conversation
Swtiched to erroring out, added a ccm test and on the dtest PR you can see the dtests changes + CI runs. |
Apologies guys, I had left behind some code from the previous commit, my bad. Removed now. |
I have added CI for this latest commit on the dtests PR to keep everything in the same place #justfyi |
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.
Changes look good to me, even though I'm not very familiarized with ccm
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.
I am also not one of the ccm code gurus on the project but I think the proposed solution is deterministic and makes the things clear. I would run it also as a comment in the slack channel to see if it might affect someone in a way but I don't see how. My only concern is it affects the CI before a release :-) Good catch of the bug
I can run it through Slack +1 |
…des without DC