From b4f652b7c1198f27a805ac29d7188a3bde1c8790 Mon Sep 17 00:00:00 2001 From: Avi Kivity Date: Sun, 5 Apr 2026 14:12:07 +0300 Subject: [PATCH] test: fix flaky test_create_ks_auth by removing bad retry timeout get_session() was passing timeout=0.1 to patient_exclusive_cql_connection and patient_cql_connection, leaving only 0.1 seconds for the retry loop in retry_till_success(). Since each connection attempt can take up to 5 seconds (connect_timeout=5), the retry loop effectively got only one attempt with no chance to retry on transient NoHostAvailable errors. Use the default timeout=30 seconds, consistent with all other callers. Fixes: SCYLLADB-1373 Closes scylladb/scylladb#29332 --- test/cluster/dtest/auth_test.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/cluster/dtest/auth_test.py b/test/cluster/dtest/auth_test.py index 3d87c38359..1e71da6000 100644 --- a/test/cluster/dtest/auth_test.py +++ b/test/cluster/dtest/auth_test.py @@ -1182,9 +1182,9 @@ class TestAuth(Tester): def get_session(self, node_idx=0, user=None, password=None, exclusive=True): node = self.cluster.nodelist()[node_idx] if exclusive: - conn = self.patient_exclusive_cql_connection(node, user=user, password=password, timeout=0.1) + conn = self.patient_exclusive_cql_connection(node, user=user, password=password) else: - conn = self.patient_cql_connection(node, user=user, password=password, timeout=0.1) + conn = self.patient_cql_connection(node, user=user, password=password) return conn def assert_permissions_listed(self, expected, session, query, include_superuser=False):