]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/mgr/selftest: handle always-on module fall out 23681/head
authorNoah Watkins <nwatkins@redhat.com>
Tue, 28 Aug 2018 20:45:56 +0000 (13:45 -0700)
committerNoah Watkins <nwatkins@redhat.com>
Tue, 28 Aug 2018 20:45:58 +0000 (13:45 -0700)
need a non-always-on module. hello doesn't work because it isn't
installed. so switch to selftest.

Signed-off-by: Noah Watkins <nwatkins@redhat.com>
qa/tasks/mgr/test_module_selftest.py

index 3851a5be867253105681382450d5fb4a5c865a34..30968118f8226ac0d22190196c0d8b6257d97bed 100644 (file)
@@ -231,15 +231,14 @@ class TestModuleSelftest(MgrTestCase):
         disabled/failed/recently-enabled modules.
         """
 
-        self._load_module("selftest")
-
         # Calling a command on a disabled module should return the proper
         # error code.
+        self._load_module("selftest")
         self.mgr_cluster.mon_manager.raw_cluster_cmd(
-            "mgr", "module", "disable", "hello")
+            "mgr", "module", "disable", "selftest")
         with self.assertRaises(CommandFailedError) as exc_raised:
             self.mgr_cluster.mon_manager.raw_cluster_cmd(
-                "hello")
+                "mgr", "self-test", "run")
 
         self.assertEqual(exc_raised.exception.exitstatus, errno.EOPNOTSUPP)
 
@@ -252,9 +251,9 @@ class TestModuleSelftest(MgrTestCase):
 
         # Enabling a module and then immediately using ones of its commands
         # should work (#21683)
+        self._load_module("selftest")
         self.mgr_cluster.mon_manager.raw_cluster_cmd(
-            "mgr", "module", "enable", "status")
-        self.mgr_cluster.mon_manager.raw_cluster_cmd("osd", "status")
+            "mgr", "self-test", "config", "get", "testkey")
 
         # Calling a command for a failed module should return the proper
         # error code.