import quincy beta 17.1.0

[ceph.git] / ceph / src / pybind / mgr / cephadm / configchecks.py

diff --git a/ceph/src/pybind/mgr/cephadm/configchecks.py b/ceph/src/pybind/mgr/cephadm/configchecks.py
index af80bd2619f15b0452c7d0daa4aeb48a4b77ec38..dc7a098277f77bb35ae1048531360b4f0a705c23 100644 (file)
--- a/ceph/src/pybind/mgr/cephadm/configchecks.py
+++ b/ceph/src/pybind/mgr/cephadm/configchecks.py
@@ -671,7 +671,7 @@ class CephadmConfigChecks:
                 self.log.warning(f"Host gather facts for {hostname} is missing kernel information")
 
             # NOTE: if daemondescription had systemd enabled state, we could check for systemd 'tampering'
-            self.host_to_role[hostname] = self.mgr.cache.get_daemon_types(hostname)
+            self.host_to_role[hostname] = list(self.mgr.cache.get_daemon_types(hostname))
 
     def run_checks(self) -> None:
         checks_enabled = self.mgr.get_module_option('config_checks_enabled')
@@ -702,12 +702,4 @@ class CephadmConfigChecks:
                 self.active_checks.append(health_check.name)
                 health_check.func()
 
-        if self.health_check_raised:
-            self.log.warning("CEPHADM checks have detected configuration anomalies")
-        else:
-            self.log.info(
-                f"CEPHADM {self.active_checks_count}/{self.defined_checks} checks enabled "
-                f"and executed ({self.skipped_checks_count} bypassed, "
-                f"{self.defined_checks - self.active_checks_count} disabled). No issues detected")
-
         self.mgr.set_health_checks(self.mgr.health_checks)