Compare commits

...

3 Commits

Author SHA1 Message Date
copilot-swe-agent[bot]
9cf774cfc6 Optimize test_downgrade_after_partial_upgrade to reduce wait_for_cql_and_get_hosts calls
Co-authored-by: patjed41 <76812270+patjed41@users.noreply.github.com>
2025-12-30 16:10:57 +00:00
copilot-swe-agent[bot]
c0f20d64ff Initial plan 2025-12-30 16:07:06 +00:00
Yaniv Kaul
ddfe3165c3 Fix for Suspicious unused loop iteration variable
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
2025-12-15 15:10:21 +02:00

View File

@@ -100,24 +100,22 @@ async def test_downgrade_after_partial_upgrade(manager: ManagerClient) -> None:
in all nodes but one, then downgrading the upgraded nodes.
"""
servers = await manager.running_servers()
cql, hosts = await manager.get_ready_cql(servers)
upgrading_servers = servers[1:] if len(servers) > 0 else []
cql = manager.cql
upgrading_hosts = hosts[1:] if len(hosts) > 0 else []
# Upgrade
for srv in upgrading_servers:
for srv, host in zip(upgrading_servers, upgrading_hosts):
await change_support_for_test_feature_and_restart(manager, [srv], enable=True)
host = (await wait_for_cql_and_get_hosts(cql, [srv], time.time() + 60))[0]
assert TEST_FEATURE_NAME in await get_supported_features(cql, host)
# There is one node that is not upgraded. The feature should not be enabled.
for srv in servers:
for srv, host in zip(servers, hosts):
assert TEST_FEATURE_NAME not in await get_enabled_features(cql, host)
# Downgrade, in reverse order
for srv in upgrading_servers[::-1]:
for srv, host in zip(reversed(upgrading_servers), reversed(upgrading_hosts)):
await change_support_for_test_feature_and_restart(manager, [srv], enable=False)
host = (await wait_for_cql_and_get_hosts(cql, [srv], time.time() + 60))[0]
assert TEST_FEATURE_NAME not in await get_supported_features(cql, host)