]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
qa: Fix test_with_health_warn_with_2_active_MDSs 65260/head
authorKotresh HR <khiremat@redhat.com>
Tue, 1 Jul 2025 11:11:37 +0000 (11:11 +0000)
committerJos Collin <jcollin@redhat.com>
Wed, 27 Aug 2025 06:49:38 +0000 (12:19 +0530)
commit8082d5974089a01c23c0aff0d2f1293feb1e3a4c
tree45fc903d73b13e06f630250f13d5d137949104e4
parent13905635f8a28607e81e6bd5dc85507fa7da0e8f
qa: Fix test_with_health_warn_with_2_active_MDSs

The test intended to validate the failure of 'mds fail'
cmd on any active mds when one of them has warning.

The commit 221700273a82658c642a282c5761c0cbb00ec5b6
(PR 61554) changes this behavior and allows 'mds fail'
on mds without the warning. The test should have always
failed with this commit. But the test never failed until
tested extensively because the test mostly generated
warnings for both active mdses. Occasionaly, the test
generated a warning on single mds and failed. So it's a
race. This patch fixes the same by changing the following.

 a. Changed the mds_cache_memory_limit to '50K' from '1K'
    as '1K' was to less and generating warning on both the mdses.
 b. Create a directory and pin it a single mds and open 400 files
    in the backend to create cache pressure on one mds.

Also, there are two tests with the same name as
'test_with_health_warn_with_2_active_MDSs' but in different classes
though. So changed the test name to
'test_with_health_warn_on_1_mds_with_2_active_MDSs' to avoid
confusion and indicate what the test actually does.

Fixes: https://tracker.ceph.com/issues/71915
Signed-off-by: Kotresh HR <khiremat@redhat.com>
(cherry picked from commit f990e7d1ea90a93b3f17a500be9178f979bf7e39)
qa/tasks/cephfs/test_admin.py