-
Notifications
You must be signed in to change notification settings - Fork 101
improvement(nemesis_tests): Make tests independant from production Nemesis #10912
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
Conversation
- Move them to separate directory - Use custom subclass hierarchies to make them less dependent on the implementation - Add NemesisRegistry tests - Fixes scylladb#10592
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.
LGTM
no backports needed? |
We did not backport nemesis registry changes so probably not. Only release where it could make sense is 2025.2, as that was branched after the registry was merged |
I see #10502 was backported to 2025.1 and as you mention it is in 2025.2 |
You are right, I remembered it wrong, sorry. I added backports to both 2025.1 and 2025.2 |
Testing
PR pre-checks (self review)
backport
labels