From: Venky Shankar <vshankar@redhat.com>
Date: Mon, 29 Mar 2021 06:44:00 +0000 (-0400)
Subject: test: disable mgr/mirroring for `test_mirroring_init_failure_with_recovery` test
X-Git-Tag: v17.1.0~2352^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=c0df8cf78d73181c7c665dd15fd862d9fb0cf5f5;p=ceph.git

test: disable mgr/mirroring for `test_mirroring_init_failure_with_recovery` test

The test simulates failures by enabling mirroring for file system
via mon interface. Doing this does not create `cephfs-mirror` index
obejct that trips the mirror daemon to report failed status via
asok interface -- so that the test can verify this.

However, mgr/mirroring module gets to know that mirroring is
enabled for this file system (via FSMap) and tried to load the
directory map, but fails to do so and throws a backtrace in the
logs. Fix this by disabling mgr/mirroring before mirroring is
enabled via mon command.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
---

diff --git a/qa/tasks/cephfs/test_mirroring.py b/qa/tasks/cephfs/test_mirroring.py
index 13c0de6cd119..9f352bedf8f3 100644
--- a/qa/tasks/cephfs/test_mirroring.py
+++ b/qa/tasks/cephfs/test_mirroring.py
@@ -740,6 +740,11 @@ class TestMirroring(CephFSTestCase):
     def test_mirroring_init_failure_with_recovery(self):
         """Test if the mirror daemon can recover from a init failure"""
 
+        # disable mgr mirroring plugin as it would try to load dir map on
+        # on mirroring enabled for a filesystem (an throw up erorrs in
+        # the logs)
+        self.disable_mirroring_module()
+
         # enable mirroring through mon interface -- this should result in the mirror daemon
         # failing to enable mirroring due to absence of `cephfs_mirorr` index object.