From: Ondrej Lichtner <olichtne(a)redhat.com>
Add just a single result instead of on per host-cpu since that can get
spammy on machines with a lot of cores...
The result message also clearly indicates that this is just a
placeholder result because an actual evaluation method wasn't
implemented.
Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com>
---
.../RecipeCommon/Perf/Measurements/BaseCPUMeasurement.py | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/lnst/RecipeCommon/Perf/Measurements/BaseCPUMeasurement.py
b/lnst/RecipeCommon/Perf/Measurements/BaseCPUMeasurement.py
index 2507f3c..f81e8df 100644
--- a/lnst/RecipeCommon/Perf/Measurements/BaseCPUMeasurement.py
+++ b/lnst/RecipeCommon/Perf/Measurements/BaseCPUMeasurement.py
@@ -64,10 +64,13 @@ class BaseCPUMeasurement(BaseMeasurement):
@classmethod
def evaluate_results(cls, recipe, results):
#TODO split off into a separate evaluator class
+ hosts = []
for result in results:
- recipe.add_result(True,
- "Base CPU evaluation for host {}, cpu {}".format(
- result.host.hostid, result.cpu))
+ if result.host.hostid not in hosts:
+ hosts.append(result.host.hostid)
+ recipe.add_result(True,
+ "CPU evaluation for results from hosts {} not implemented"
+ .format(hosts))
@classmethod
def _divide_results_by_host(cls, results):
--
2.21.0