From c94ada69e80d7a1ddfbd2de2b13086d57a6fdfcd Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Wed, 6 Jun 2018 21:56:38 +0200 Subject: [PATCH] tests: improve mds tests the expected number of mds daemon consist of number of daemons that are 'up' + number of daemons 'up:standby'. Signed-off-by: Guillaume Abrioux --- tests/functional/tests/mds/test_mds.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/tests/functional/tests/mds/test_mds.py b/tests/functional/tests/mds/test_mds.py index 3bbc5fc5f..834382177 100644 --- a/tests/functional/tests/mds/test_mds.py +++ b/tests/functional/tests/mds/test_mds.py @@ -23,9 +23,9 @@ class TestMDSs(object): def test_mds_is_up(self, node, host): hostname = node["vars"]["inventory_hostname"] cmd = "sudo ceph --name client.bootstrap-mds --keyring /var/lib/ceph/bootstrap-mds/{cluster}.keyring --cluster={cluster} --connect-timeout 5 -f json -s".format(cluster=node['cluster_name']) - output = host.check_output(cmd) - daemons = json.loads(output)["fsmap"]["by_rank"][0]["name"] - assert hostname in daemons + cluster_status = json.loads(host.check_output(cmd)) + + assert (cluster_status['fsmap'].get('up', 0) + cluster_status['fsmap'].get('up:standby', 0)) == len(node["vars"]["groups"]["mdss"]) @pytest.mark.docker def test_docker_mds_is_up(self, node, host): @@ -34,7 +34,6 @@ class TestMDSs(object): hostname=node["vars"]["inventory_hostname"], cluster=node["cluster_name"] ) - num_mdss = len(host.ansible.get_variables()["groups"]["mdss"]) - output_raw = host.check_output(cmd) - output_json = json.loads(output_raw) - assert output_json['fsmap']['up'] and output_json['fsmap']['in'] == num_mdss \ No newline at end of file + cluster_status = json.loads(host.check_output(cmd)) + + assert (cluster_status['fsmap'].get('up', 0) + cluster_status['fsmap'].get('up:standby', 0)) == len(node["vars"]["groups"]["mdss"]) -- 2.39.5