]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
qa: Fix test_with_health_warn_with_2_active_MDSs
authorKotresh HR <khiremat@redhat.com>
Tue, 1 Jul 2025 11:11:37 +0000 (11:11 +0000)
committerJos Collin <jcollin@redhat.com>
Fri, 3 Oct 2025 02:35:17 +0000 (08:05 +0530)
commit8873506c213d8ac0df64d2811bdca931d39bcf43
tree86ed2152fcbd44a1a168e968d50f981ae97c5c62
parent17b07d77fa8cf061eef61ad9dbc2c54e3ef49e73
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