From f0c283542dca05e5672697ebf25caa81a831c9bb Mon Sep 17 00:00:00 2001 From: Venky Shankar Date: Sat, 7 Oct 2023 20:42:29 +0530 Subject: [PATCH] qa: lower mds_session_metadata_threshold for tests ... and increase the number of files that are created so as to hit the threshold with a high probability. Fixes: http://tracker.ceph.com/issues/62873 Signed-off-by: Venky Shankar (cherry picked from commit 92200d9d10d30ed35f640fd1120be65577a3ce64) --- qa/tasks/cephfs/test_client_limits.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/qa/tasks/cephfs/test_client_limits.py b/qa/tasks/cephfs/test_client_limits.py index 70c077f16c58b..c94dbc6af197d 100644 --- a/qa/tasks/cephfs/test_client_limits.py +++ b/qa/tasks/cephfs/test_client_limits.py @@ -248,15 +248,14 @@ class TestClientLimits(CephFSTestCase): # Wait for the health warnings. Assume mds can handle 10 request per second at least self.wait_for_health("MDS_CLIENT_OLDEST_TID", max_requests // 10, check_in_detail=str(self.mount_a.client_id)) - # why a multiplier of 20, you may ask - I arrieved at this from some debugs - # that I put when testing the fix in a vstart cluster where its a ratio of - # encoded session information to the number of completed requests. - self.config_set('mds', 'mds_session_metadata_threshold', max_requests*20); + # set the threshold low so that it has a high probability of + # hitting. + self.config_set('mds', 'mds_session_metadata_threshold', 5000); - # Create a few more files synchronously. This would hit the session metadata threshold + # Create lot many files synchronously. This would hit the session metadata threshold # causing the client to get blocklisted. with self.assertRaises(CommandFailedError): - self.mount_a.create_n_files("testdir/file2", 20, True) + self.mount_a.create_n_files("testdir/file2", 100000, True) self.mds_cluster.is_addr_blocklisted(self.mount_a.get_global_addr()) # the mds should bump up the relevant perf counter -- 2.39.5