diff --git a/selftest/test/test_cpu_sentry.py b/selftest/test/test_cpu_sentry.py index 47e366b7633dd38d3291c5444b9e81789223c776..c9af1842de3e705ea72472fd13bc42203d51ddf0 100644 --- a/selftest/test/test_cpu_sentry.py +++ b/selftest/test/test_cpu_sentry.py @@ -20,6 +20,7 @@ class TestCaseCpuSentry: ("1-", []), ("-1", []), ("", []), + (" ", []), ("1-3,5-8", [1,2,3,5,6,7,8]), ("1-3,8", [1,2,3,8]), ("11", [11]), @@ -53,6 +54,7 @@ class TestCaseCpuSentry: ("0, 2-4", False), ("1-3-6", False), ("", False), + (" ", False), ("xxx", False), ("1-3,,", False), ("1-3,,6", False), diff --git a/src/sentryPlugins/cpu_sentry/cpu_sentry.py b/src/sentryPlugins/cpu_sentry/cpu_sentry.py index 0c66fd28e59c711425df5ad4e5231cd0bfa6581f..e7a34ad9b86b0108536517dc6770755a2b092efa 100644 --- a/src/sentryPlugins/cpu_sentry/cpu_sentry.py +++ b/src/sentryPlugins/cpu_sentry/cpu_sentry.py @@ -142,6 +142,12 @@ class CpuSentry: self.send_result["details"]["code"] = 1002 self.send_result["result"] = ResultLevel.MINOR_ALM self.send_result["details"]["msg"] = "Some CPUs are faulty. The faulty cores are isolated successfully." + else: + found_fault_cores_set = set(found_fault_cores_list) + isolated_cpu_set = set(CpuSentry.cpu_format_convert_to_list(self.send_result["details"]["isolated_cpu_list"])) + self.send_result["details"]["code"] = 1002 + self.send_result["result"] = ResultLevel.MINOR_ALM + self.send_result["details"]["msg"] = "Some cores are isolated successfully and some cores ({}) fail to be isolated.".format(list(found_fault_cores_set - isolated_cpu_set)) def cpu_report_result(self): """report result to sysSentry"""