scylla-cluster-tests icon indicating copy to clipboard operation
scylla-cluster-tests copied to clipboard

Published 20 hours ago • verified publisher icon scylladb

improvement(fill_db_data): Remove 30s sleep of waiting for schema agreement

Open roydahan opened this issue 10 months ago • 1 comments

Now that all of our recent releases contains raft with consistent scehma management, this sleep isn't needed anymore.

Test: rolling upgrades.

Fixes: #9660

Testing

  • [ ] https://jenkins.scylladb.com/job/scylla-staging/job/roy/job/rolling-upgrade-ami-test/2/

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)

roydahan avatar Jan 13 '25 15:01 roydahan

@roydahan seems test was failing cause it was missing the extra timeout for the truncate operation.

maybe for the safe side, rebase this PR, and try it again ?

fruch avatar Jan 15 '25 21:01 fruch

Testing here

k0machi avatar Jun 03 '25 12:06 k0machi

Testing here

Failure seems to be unrelated to change, getting events about oversized_allocation

k0machi avatar Jun 06 '25 15:06 k0machi