This is an automated email from the ASF dual-hosted git repository.

samt pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/cassandra-dtest.git


The following commit(s) were added to refs/heads/trunk by this push:
     new f8be8502 Ignore harmless error message if system_auth reads occur 
during an rf change
f8be8502 is described below

commit f8be85023ad75ead695f6f014d3cc391fbce43b2
Author: Sam Tunnicliffe <[email protected]>
AuthorDate: Thu May 30 15:43:00 2024 +0100

    Ignore harmless error message if system_auth reads occur during an rf change
    
    Patch by Sam Tunnicliffe; reviewed by Brandon Williams for CASSANDRA-19217
---
 auth_test.py | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/auth_test.py b/auth_test.py
index ea1ad31a..1f0dec10 100644
--- a/auth_test.py
+++ b/auth_test.py
@@ -2792,6 +2792,19 @@ class TestAuthUnavailable(AbstractTestAuth):
     * release regimes.
     """
 
+    @pytest.fixture(autouse=True)
+    def fixture_add_additional_log_patterns(self, fixture_dtest_setup):
+        fixture_dtest_setup.ignore_log_patterns = (
+            # This occurs if a client interaction which results in a specific 
type of query on system_auth, happens to
+            # run concurrently with changing the rf from 1 to 2. For example, 
a query to read the default `cassandra`
+            # superuser will execute at CL.QUORUM. If this query is issued 
before the rf change, the replica plan will
+            # contain only 1 node. Should the rf change be enacted before the 
response is processed by the coordinator,
+            # that original replica plan will no longer be sufficient to 
satisfy the consistency level, as now QUORUM is
+            # (2/2 + 1) = 2. This does not affect the actual test as the 
client will retry in this case and as the
+            # placements remain stable after the initial rf change, subsequent 
reads will succeed.
+            r'.* for keyspace system_auth at QUORUM the ring has changed .* in 
a way that would make responses violate the consistency level',
+        )
+
     def test_authentication_handle_unavailable(self):
         """
         * Launch a two node cluster with role/permissions cache disabled


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to