From: Zack Cerza Date: Thu, 29 Mar 2018 00:12:42 +0000 (-0600) Subject: ceph-graphite: Fix test_metrics_present X-Git-Tag: v2.0~37^2~8 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=930f1a78baa8a3e4796a6d0afc2a4db01eccd76f;p=cephmetrics.git ceph-graphite: Fix test_metrics_present If a cluster changes over time, the test broke. When looking at stored metrics, assert that our inventory hosts are a subset of the stored metrics as opposed to doing an equality check. Signed-off-by: Zack Cerza --- diff --git a/ansible/roles/ceph-graphite/tests/test_graphite.py b/ansible/roles/ceph-graphite/tests/test_graphite.py index 5121987..e15a7d3 100644 --- a/ansible/roles/ceph-graphite/tests/test_graphite.py +++ b/ansible/roles/ceph-graphite/tests/test_graphite.py @@ -90,6 +90,5 @@ class TestGraphite(object): def extract_hostname(fragment): return fragment['text'] - metric_hosts = map(extract_hostname, obj) - assert sorted(map(lambda s: s.split('.')[0], ceph_hosts)) == \ - sorted(metric_hosts) + metric_hosts = set(map(extract_hostname, obj)) + assert set(map(lambda s: s.split('.')[0], ceph_hosts)).issubset(metric_hosts)