From 394820622512091cb1693aaff99ea3a4c163219c Mon Sep 17 00:00:00 2001
From: "Joshua M. Boniface" <joshua@boniface.me>
Date: Sun, 3 Oct 2021 15:06:18 -0400
Subject: [PATCH] Tweak fio tests for benchmarks

1. Remove ramp_time as this was giving very strange results.

2. Up the runtime to 75 seconds to compensate.

3. Print the fio command to the console to validate.
---
 api-daemon/pvcapid/benchmark.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/api-daemon/pvcapid/benchmark.py b/api-daemon/pvcapid/benchmark.py
index 1569e3a6..c54d800f 100755
--- a/api-daemon/pvcapid/benchmark.py
+++ b/api-daemon/pvcapid/benchmark.py
@@ -244,8 +244,7 @@ def run_benchmark(self, pool):
                 --randrepeat=1 \
                 --numjobs=1 \
                 --time_based \
-                --runtime=60 \
-                --ramp_time=15 \
+                --runtime=75 \
                 --group_reporting \
                 --iodepth={iodepth} \
                 --bs={bs} \
@@ -258,6 +257,7 @@ def run_benchmark(self, pool):
             bs=test_matrix[test]['bs'],
             rw=test_matrix[test]['rw'])
 
+        print("Running fio job: {}".format(' '.join(fio_cmd.split())))
         retcode, stdout, stderr = pvc_common.run_os_command(fio_cmd)
         if retcode:
             raise BenchmarkError("Failed to run fio test: {}".format(stderr), job_name=job_name, db_conn=db_conn, db_cur=db_cur, zkhandler=zkhandler)