From 0d7ecb18e33b05bb957f4da7c43199086c7d8ab9 Mon Sep 17 00:00:00 2001 From: Paul Cuzner Date: Fri, 19 Feb 2021 12:52:24 +1300 Subject: [PATCH] mgr/cephadm:unit test added for nics better than most checks that we're not raising a healthcheck for a host if it's nic speed it better than the rest! Signed-off-by: Paul Cuzner --- src/pybind/mgr/cephadm/tests/test_configchecks.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/src/pybind/mgr/cephadm/tests/test_configchecks.py b/src/pybind/mgr/cephadm/tests/test_configchecks.py index 534670d829d..051b017a69e 100644 --- a/src/pybind/mgr/cephadm/tests/test_configchecks.py +++ b/src/pybind/mgr/cephadm/tests/test_configchecks.py @@ -419,6 +419,20 @@ class TestConfigCheck: assert mgr.health_checks['CEPHADM_CHECK_LINKSPEED']['detail'][0] == \ 'host node-1.ceph.com(eth0) has linkspeed of 100 on 10.9.64.0/24, NICs on other hosts use 1000' + def test_super_linkspeed_single(self, mgr): + + bad_node = mgr.cache.facts['node-1.ceph.com'] + bad_node['interfaces']['eth0']['speed'] = 10000 + + checker = CephadmConfigChecks(mgr) + checker.cluster_network_list = [] + checker.public_network_list = ['10.9.64.0/24'] + + checker.run_checks() + logger.info(json.dumps(mgr.health_checks)) + logger.info(checker.subnet_lookup) + assert not mgr.health_checks + def test_release_mismatch_single(self, mgr): mgr.version_overrides = { -- 2.47.3