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

fix(scylla-bench large-partitions): Downgrade scylla-bench to a stable version #4457

Merged
merged 1 commit into from
Feb 21, 2022

Conversation

yarongilor
Copy link
Contributor

@yarongilor yarongilor commented Feb 15, 2022

scylla-bench latest version of 0.1.8 is not stable.
Therefore downgrading it on large-partitions relevant longevities.

Trello: https://trello.com/c/8U4dGEEI

PR pre-checks (self review)

  • I followed KISS principle and best practices
  • I didn't leave commented-out/debugging code
  • I added the relevant backport labels
  • New configuration option are added and documented (in sdcm/sct_config.py)
  • I have added tests to cover my changes (Infrastructure only - under unit-test/ folder)
  • All new and existing unit tests passed (CI)
  • I have updated the Readme/doc folder accordingly (if needed)

@yarongilor yarongilor force-pushed the downgrade_scylla_bench_version branch 2 times, most recently from 8419d40 to 5306cb6 Compare February 15, 2022 17:40
@yarongilor yarongilor requested review from fgelcer and removed request for bentsi and psarna February 17, 2022 08:32
fgelcer
fgelcer previously approved these changes Feb 17, 2022
Copy link
Contributor

@fgelcer fgelcer left a comment

Choose a reason for hiding this comment

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

until s-b issues will be fixed, and it will become stable again, this downgrade is to the last known "good" version:
one of the issues is this one:
scylladb/scylla-bench#91

@@ -45,3 +45,6 @@ space_node_threshold: 644245094
run_fullscan: '{"ks_cf": "scylla_bench.test", "interval": 10}' # 'ks.cf|random, interval(min)'

run_full_partition_scan: '{"ks_cf": "scylla_bench.test", "interval": 2, "pk_name":"pk", "rows_count": 5000, "validate_data": "false"}' # 'ks.cf, interval(sec), partition-key name, number-of-rows-per-partition, validate reversed query output'

# Temporarily downgrade scylla_bench to a stable version
scylla_bench_version: v0.1.3
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think it happened here as well.

Copy link
Contributor Author

@yarongilor yarongilor Feb 21, 2022

Choose a reason for hiding this comment

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

right, only in this large-partitions longevity, no s-b issues found. can keep the latest version for it.

…e version

	scylla-bench latest version of 0.1.8 is not stable.
	Therefore downgrading it on large-partitions relevant longevities.
Copy link
Contributor

@fgelcer fgelcer left a comment

Choose a reason for hiding this comment

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

LGTM

@roydahan roydahan merged commit 214326a into scylladb:master Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants