From 3fdad23e634d035010d2e85d5527d8a68d39b751 Mon Sep 17 00:00:00 2001 From: Lucian Petrut Date: Tue, 7 Feb 2023 15:13:54 +0200 Subject: [PATCH] quincy: qa: update test_rbd_wnbd.py to report latency in seconds At the moment, the latency results are reported in nanoseconds. In order to improve readability, we'll convert it to seconds. While at it, we'll fix the fio duration report, which we're wrongfully dividing by 1000 twice. Signed-off-by: Lucian Petrut --- qa/workunits/windows/test_rbd_wnbd.py | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/qa/workunits/windows/test_rbd_wnbd.py b/qa/workunits/windows/test_rbd_wnbd.py index c749957cf655..f22810e2ea14 100644 --- a/qa/workunits/windows/test_rbd_wnbd.py +++ b/qa/workunits/windows/test_rbd_wnbd.py @@ -649,7 +649,7 @@ class RbdFioTest(RbdTest): s['median'], s['std_dev'], s['max_90'], s['min_90'], 'N/A']) - s = array_stats([float(i["runtime"]) / 1000 for i in op_data]) + s = array_stats([float(i["runtime"]) for i in op_data]) table.add_row(["duration (s)", s['min'], s['max'], s['mean'], s['median'], s['std_dev'], @@ -681,10 +681,16 @@ class RbdFioTest(RbdTest): if len(op_data) else 0) clat_10 = array_stats([i["clat_ns_10"] for i in op_data]) clat_90 = array_stats([i["clat_ns_90"] for i in op_data]) - table.add_row(["completion latency (ns)", - clat_min['min'], clat_max['max'], clat_mean['mean'], - clat_mean['median'], clat_stddev, - clat_10['mean'], clat_90['mean'], clat_mean['sum']]) + # For convenience, we'll convert it from ns to seconds. + table.add_row(["completion latency (s)", + clat_min['min'] / 1e+9, + clat_max['max'] / 1e+9, + clat_mean['mean'] / 1e+9, + clat_mean['median'] / 1e+9, + clat_stddev / 1e+9, + clat_10['mean'] / 1e+9, + clat_90['mean'] / 1e+9, + clat_mean['sum'] / 1e+9]) print(table) -- 2.47.3