Skip to content

improvement(upgrade): check limited voters feature in latte upgrade test #12291

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 22, 2025
Merged

improvement(upgrade): check limited voters feature in latte upgrade test #12291

merged 1 commit into from
Oct 22, 2025

Conversation

@vponomaryov
Copy link
Contributor

@vponomaryov vponomaryov commented Oct 22, 2025 •
edited
Loading

Run validate_limited_voters_feature_enabled method
in the test_cluster_upgrade_latency_regression upgrade test.

Ref: #10853

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)

@vponomaryov
improvement(upgrade): check limited voters feature in latte upgrade test
dd70cb1 
Run 'validate_limited_voters_feature_enabled' method
in the 'test_cluster_upgrade_latency_regression' upgrade test.

Ref: scylladb#10853
@vponomaryov
Copy link
Contributor Author

It's workability was tested throughout the following task:

@vponomaryov vponomaryov requested review from a team and aleksbykov October 22, 2025 17:06
fruch
fruch approved these changes Oct 22, 2025
View reviewed changes
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

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

LGTM

@fruch fruch merged commit 08530e3 into scylladb:master Oct 22, 2025
13 of 14 checks passed
This was referenced Oct 22, 2025
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fruch fruch fruch approved these changes

@aleksbykov aleksbykov Awaiting requested review from aleksbykov

Assignees

@vponomaryov vponomaryov

Labels

backport/perf-v17-done backport/2025.3-done backport/2025.4-done promoted-to-master

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vponomaryov @fruch @scylladbbot