Skip to content

Commit

Permalink
feature(manager): check system_auth ks repair only for raft topology
Browse files Browse the repository at this point in the history
There is no system_auth ks in Scylla with enabled raft topology. Thus,
the test should check whether consistent topology changes are enabled
or not to form the valid list of repaired keyspaces.
  • Loading branch information
mikliapko authored and fruch committed Jun 25, 2024
1 parent 5681a09 commit 486b5a8
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions mgmt_cli_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -841,8 +841,10 @@ def test_repair_multiple_keyspace_types(self): # pylint: disable=invalid-name
self.log.info('Task: {} is done.'.format(repair_task.id))
self.log.debug("sctool version is : {}".format(manager_tool.sctool.version))

expected_keyspaces_to_be_repaired = ["system_auth", "system_distributed", # pylint: disable=invalid-name
self.NETWORKSTRATEGY_KEYSPACE_NAME]
expected_keyspaces_to_be_repaired = ["system_distributed", self.NETWORKSTRATEGY_KEYSPACE_NAME]
if not self.db_cluster.nodes[0].raft.is_consistent_topology_changes_enabled:
expected_keyspaces_to_be_repaired.append("system_auth")
self.log.debug("Keyspaces expected to be repaired: {}".format(expected_keyspaces_to_be_repaired))
per_keyspace_progress = repair_task.per_keyspace_progress
self.log.info("Looking in the repair output for all of the required keyspaces")
for keyspace_name in expected_keyspaces_to_be_repaired:
Expand Down

0 comments on commit 486b5a8

Please sign in to comment.