diff options
| author | Venky Shankar <vshankar@redhat.com> | 2023-10-09 14:19:30 +0200 |
|---|---|---|
| committer | Venky Shankar <vshankar@redhat.com> | 2023-10-09 14:19:30 +0200 |
| commit | 1568321c17f07149aa2bfb31fc58235ce571c3a3 (patch) | |
| tree | 9ed9992edda63ec87cf4d721554e1bb81147188f | |
| parent | Merge pull request #53817 from cloudbehl/active-alert-filter (diff) | |
| parent | qa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_ev... (diff) | |
| download | ceph-1568321c17f07149aa2bfb31fc58235ce571c3a3.tar.xz ceph-1568321c17f07149aa2bfb31fc58235ce571c3a3.zip | |
Merge PR #53873 into main
* refs/pull/53873/head:
qa: typo fix when checking for perf counter - s/md_thresh_evicted/mdthresh_evicted
qa: lower mds_session_metadata_threshold for tests
Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Rishabh Dave <ridave@redhat.com>
Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
| -rw-r--r-- | .githubmap | 1 | ||||
| -rw-r--r-- | qa/tasks/cephfs/test_client_limits.py | 13 |
2 files changed, 7 insertions, 7 deletions
diff --git a/.githubmap b/.githubmap index b5d44763134..cf5f15c130a 100644 --- a/.githubmap +++ b/.githubmap @@ -175,3 +175,4 @@ phlogistonjohn John Mulligan <jmulligan@redhat.com> baergj Joshua Baergen <jbaergen@digitalocean.com> zmc Zack Cerza <zack@redhat.com> robbat2 Robin H. Johnson <robbat2@orbis-terrarum.net> +leonid-s-usov Leonid Usov <leonid.usov@ibm.com> diff --git a/qa/tasks/cephfs/test_client_limits.py b/qa/tasks/cephfs/test_client_limits.py index 7f3a91b96f9..6961f4c7218 100644 --- a/qa/tasks/cephfs/test_client_limits.py +++ b/qa/tasks/cephfs/test_client_limits.py @@ -272,20 +272,19 @@ 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 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') |