From d9309b57e3c12579f64939ee78161f7e99a062c8 Mon Sep 17 00:00:00 2001 From: John Spray Date: Tue, 29 Mar 2016 12:11:18 +0100 Subject: [PATCH] tasks/cephfs: update failover test for standby changes Previously, an MDS could be used as a replay even if it had "standby replay = false", this is changed now so that replays will only happen explicitly. Signed-off-by: John Spray --- tasks/cephfs/test_failover.py | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/tasks/cephfs/test_failover.py b/tasks/cephfs/test_failover.py index 6fa209cf6b2f5..5183beb95f7de 100644 --- a/tasks/cephfs/test_failover.py +++ b/tasks/cephfs/test_failover.py @@ -365,17 +365,8 @@ class TestMultiFilesystems(CephFSTestCase): self.mds_cluster.mds_restart(mds_c) self.wait_for_daemon_start([mds_a, mds_c]) - # See that they take up standby replay roles - # mds_a should be in filesystem alpha following mds_b - info_a = get_info_by_name(fs_a, mds_a) - self.assertEqual(info_a['state'], "up:standby-replay") - self.assertEqual(info_a['standby_for_name'], mds_b) - self.assertEqual(info_a['rank'], 0) - # mds_c should be in filesystem alpha following mds_d - info_c = get_info_by_name(fs_b, mds_c) - self.assertEqual(info_c['state'], "up:standby-replay") - self.assertEqual(info_c['standby_for_name'], mds_d) - self.assertEqual(info_c['rank'], 0) + self.assertEqual(set(self.mds_cluster.get_standby_daemons()), + {mds_a, mds_c}) def test_standby_for_rank(self): use_daemons = sorted(self.mds_cluster.mds_ids[0:4]) -- 2.39.5