From: Brad Hubbard Date: Tue, 20 Jul 2021 02:58:15 +0000 (+1000) Subject: qa/tasks/mgr/test_insights: Remove test for persistent checks X-Git-Tag: v16.2.6~36^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=307ce65a28962e5ff4461ae3feb7e93574e7f53e;p=ceph.git qa/tasks/mgr/test_insights: Remove test for persistent checks This test makes no sense if we are no longer persisting the store. Signed-off-by: Brad Hubbard (cherry picked from commit 32d1cca2d9b606915c590f52d61856ee401fb4fc) --- diff --git a/qa/tasks/mgr/test_insights.py b/qa/tasks/mgr/test_insights.py index 9f952152c695d..aa2548881d367 100644 --- a/qa/tasks/mgr/test_insights.py +++ b/qa/tasks/mgr/test_insights.py @@ -149,17 +149,6 @@ class TestInsights(MgrTestCase): active_id = self.mgr_cluster.get_active_id() self.mgr_cluster.mgr_restart(active_id) - # ensure that at least one of the checks is present after the restart. - # we don't for them all to be present because "earlier" checks may not - # have sat in memory long enough to be flushed. - all_missing = True - report = self._insights() - for check in check_names: - if check in report["health"]["history"]["checks"]: - all_missing = False - break - self.assertFalse(all_missing) - # pruning really removes history self.mgr_cluster.mon_manager.raw_cluster_cmd_result( "insights", "prune-health", "0")