Skip to content
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

improvement(nemesis): allow specify different seeds for parallel nemesis #9992

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

soyacz
Copy link
Contributor

@soyacz soyacz commented Feb 4, 2025

When running parallel nemesis all take the same seed. Sometimes one might specify different ones for each.

This commit allows to specify seeds as a list (or space separated ints) so each nemesis will get own seed.
It works by cycling through nemesis_seed list when defining nemesis arguments.

closes: #9333

Testing

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Reminders

  • Add New configuration option and document them (in sdcm/sct_config.py)
  • Add unit tests to cover my changes (under unit-test/ folder)
  • Update the Readme/doc folder relevant to this change (if needed)

When running parallel nemesis all take the same seed. Sometimes one
might specify different ones for each.

This commit allows to specify seeds as a list (or space separated ints)
so each nemesis will get own seed.
It works by cycling through nemesis_seed list when defining nemesis
arguments.

closes: scylladb#9333
@soyacz soyacz added the backport/none Backport is not required label Feb 4, 2025
@soyacz soyacz requested review from fruch and a team February 4, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/none Backport is not required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot set nemesis_seed for each parallel nemesis
1 participant