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

test(performance): add latency with nemesis and disable RBNO #9440

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

Conversation

juliayakovlev
Copy link
Contributor

Add 'scylla-enterprise-perf-regression-latency-650gb-with-nemesis' test with RBNO disabled

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)

@juliayakovlev
Copy link
Contributor Author

@roydahan @fruch
will we trigger this test with RBNO disabled automatically? If yes - weekly or once in 3 weeks?

Add 'scylla-enterprise-perf-regression-latency-650gb-with-nemesis' test with RBNO
disabled
perfRegressionParallelPipeline(
backend: "aws",
test_name: "performance_regression_test.PerformanceRegressionTest",
test_config: """["test-cases/performance/perf-regression-latency-650gb-with-nemesis.yaml", "configurations/tablets_disabled.yaml", "configurations/disable_kms.yaml","configurations/disable_rbno.yaml"]""",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tablets disabled - got me rethinking of it.
It was mostly relevant for us only for the runs of 2024.1, possibly for 2024.2.
However, longer term we don't want to run with tablets disabled.

However, with tablets I think this test is not needed (it doesn't use RBNO AFAIK, does it?).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I found RBNO is used when tablets enabled

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in the 2022.2 and 2023.1 versions RBNO is enabled by default for replace operation only

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@roydahan this question should go to tablet team.

@bhalevy @asias, is disabling RBNO + enabling tablets makes any sense testing (and specifically operation performance test) ?

(the only thing I can think of is, some keyspace are tablets some not, and one want to use streaming)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants