Skip to content

Commit

Permalink
fix(upgrades): replace sstabledump with scylla sstable dump-data
Browse files Browse the repository at this point in the history
cause of new sstable uuid identifer feature the old `sstabledump`
can't currently work with those sstables

Ref: scylladb/scylla-tools-java#333
  • Loading branch information
fruch committed Aug 6, 2023
1 parent 9b785b7 commit 4515fdb
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion upgrade_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -697,7 +697,9 @@ def test_rolling_upgrade(self): # pylint: disable=too-many-locals,too-many-stat
InfoEvent(message='Starting sstabledump to verify correctness of sstables').publish()
self.db_cluster.nodes[0].remoter.run(
'for i in `sudo find /var/lib/scylla/data/keyspace_complex/ -type f |grep -v manifest.json |'
'grep -v snapshots |head -n 1`; do echo $i; sudo sstabledump $i 1>/tmp/sstabledump.output || '
'grep -v snapshots |head -n 1`; do echo $i; '
f'sudo {self.db_cluster.nodes[0].add_install_prefix("/usr/bin/scylla")} sstable dump-data '
f'--sstables $i 1>/tmp/sstabledump.output || '
'exit 1; done', verbose=True)

InfoEvent(message='Step8 - Run stress and verify after upgrading entire cluster').publish()
Expand Down

0 comments on commit 4515fdb

Please sign in to comment.