From faba13d2b7486bffd7c1313f953d6febedc3d0a3 Mon Sep 17 00:00:00 2001 From: Alex Date: Sun, 5 Apr 2026 19:46:15 +0300 Subject: [PATCH] test/auth_cluster: cover empty legacy table in service level upgrade Add a cluster test that upgrades to raft topology with an empty legacy `system_distributed.service_levels` table and verifies that the migration still marks `service_level_version` as `2`. --- .../auth_cluster/test_raft_service_levels.py | 21 +++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/test/cluster/auth_cluster/test_raft_service_levels.py b/test/cluster/auth_cluster/test_raft_service_levels.py index ae837c42f0..53f014a770 100644 --- a/test/cluster/auth_cluster/test_raft_service_levels.py +++ b/test/cluster/auth_cluster/test_raft_service_levels.py @@ -114,6 +114,27 @@ async def test_service_levels_upgrade(request, manager: ManagerClient, build_mod result_with_sl_v2 = await cql.run_async(f"SELECT service_level FROM system.service_levels_v2") assert set([sl.service_level for sl in result_with_sl_v2]) == set(sls + [DRIVER_SL_NAME] + [sl_v2]) +@pytest.mark.asyncio +async def test_service_levels_upgrade_with_empty_legacy_table(manager: ManagerClient): + cfg = {**auth_config, "force_gossip_topology_changes": True, "tablets_mode_for_new_keyspaces": "disabled"} + + servers = [await manager.server_add(config=cfg)] + cfg.pop("force_gossip_topology_changes") + servers += [await manager.server_add(config=cfg) for _ in range(2)] + + cql = manager.get_cql() + assert cql + hosts = await wait_for_cql_and_get_hosts(cql, servers, time.time() + 60) + + rows = await cql.run_async("SELECT service_level FROM system_distributed.service_levels") + assert list(rows) == [] + + await manager.api.upgrade_to_raft_topology(hosts[0].address) + await asyncio.gather(*(wait_until_topology_upgrade_finishes(manager, h.address, time.time() + 60) for h in hosts)) + + sl_version = await cql.run_async("SELECT value FROM system.scylla_local WHERE key = 'service_level_version'") + assert sl_version[0].value == "2" + @pytest.mark.asyncio async def test_service_levels_work_during_recovery(manager: ManagerClient): # FIXME: move this test to the Raft-based recovery procedure or remove it if unneeded.