diff --git a/.githubmap b/.githubmap index b5d447631344a..cf5f15c130a7e 100644 --- a/.githubmap +++ b/.githubmap @@ -175,3 +175,4 @@ phlogistonjohn John Mulligan baergj Joshua Baergen zmc Zack Cerza robbat2 Robin H. Johnson +leonid-s-usov Leonid Usov diff --git a/qa/tasks/cephfs/test_client_limits.py b/qa/tasks/cephfs/test_client_limits.py index 7f3a91b96f9bd..6961f4c72183b 100644 --- a/qa/tasks/cephfs/test_client_limits.py +++ b/qa/tasks/cephfs/test_client_limits.py @@ -272,20 +272,19 @@ def test_client_blocklisted_oldest_tid(self): # 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 pd = self.perf_dump() - self.assertGreater(pd['mds_sessions']['md_thresh_evicted'], 0) + self.assertGreater(pd['mds_sessions']['mdthresh_evicted'], 0) # reset the config self.config_set('client', 'client inject fixed oldest tid', 'false')