Skip to content

Commit

Permalink
fix(azure): make upgrade work on Azure
Browse files Browse the repository at this point in the history
Skip the sstables validation after the DB nodes upgrades
due to the known issue [1].

[1] scylladb/scylla-tools-java#338
  • Loading branch information
vponomaryov committed Mar 7, 2024
1 parent b7636f8 commit 193eab0
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions upgrade_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -692,11 +692,12 @@ def test_rolling_upgrade(self): # pylint: disable=too-many-locals,too-many-stat
assert all(tables_upgraded), "Failed to upgrade the sstable format {}".format(tables_upgraded)

# Verify sstabledump
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 || '
'exit 1; done', verbose=True)
if self.params.get("cluster_backend") != "azure":
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 || '
'exit 1; done', verbose=True)

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

0 comments on commit 193eab0

Please sign in to comment.