From 8290a8a2e04b559e580620a1d291945aeb8343c9 Mon Sep 17 00:00:00 2001 From: Egor Lazarchuk Date: Wed, 14 May 2025 15:03:09 +0100 Subject: [PATCH] chore(ci): don't store latency for libaio Latency for block device only makes sence if we use `psync` fio backend. Signed-off-by: Egor Lazarchuk --- .../integration_tests/performance/test_block_ab.py | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/tests/integration_tests/performance/test_block_ab.py b/tests/integration_tests/performance/test_block_ab.py index 2068549b3d8..dfd0728084a 100644 --- a/tests/integration_tests/performance/test_block_ab.py +++ b/tests/integration_tests/performance/test_block_ab.py @@ -67,14 +67,18 @@ def run_fio(microvm, mode, block_size, test_output_dir, fio_engine="libaio"): ) # Instruct fio to pin one worker per vcpu .with_arg("--cpus_allowed_policy=split") - .with_arg(f"--write_bw_log={mode}") - .with_arg(f"--write_lat_log={mode}") .with_arg("--log_avg_msec=1000") + .with_arg(f"--write_bw_log={mode}") .with_arg("--output-format=json+") .with_arg("--output=/tmp/fio.json") - .build() ) + # Latency measurements only make sence for psync engine + if fio_engine == "psync": + cmd = cmd.with_arg(f"--write_lat_log={mode}") + + cmd = cmd.build() + prepare_microvm_for_test(microvm) # Start the CPU load monitor. @@ -110,7 +114,9 @@ def process_fio_log_files(root_dir, logs_glob): for pathname in glob.glob(logs_glob, root_dir=root_dir) ] - assert data, "no log files found!" + # If not data found, there is nothing to iterate over + if not data: + return [], [] for tup in zip(*data): read_values = []