Skip to content

Commit 9fb5a46

Browse files
committed
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 <yegorlz@amazon.co.uk>
1 parent 4ed5ccc commit 9fb5a46

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

tests/integration_tests/performance/test_block_ab.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -67,14 +67,18 @@ def run_fio(microvm, mode, block_size, test_output_dir, fio_engine="libaio"):
6767
)
6868
# Instruct fio to pin one worker per vcpu
6969
.with_arg("--cpus_allowed_policy=split")
70-
.with_arg(f"--write_bw_log={mode}")
71-
.with_arg(f"--write_lat_log={mode}")
7270
.with_arg("--log_avg_msec=1000")
71+
.with_arg(f"--write_bw_log={mode}")
7372
.with_arg("--output-format=json+")
7473
.with_arg("--output=/tmp/fio.json")
75-
.build()
7674
)
7775

76+
# Latency measurements only make sence for psync engine
77+
if fio_engine == "psync":
78+
cmd = cmd.with_arg(f"--write_lat_log={mode}")
79+
80+
cmd = cmd.build()
81+
7882
prepare_microvm_for_test(microvm)
7983

8084
# Start the CPU load monitor.
@@ -110,7 +114,9 @@ def process_fio_log_files(root_dir, logs_glob):
110114
for pathname in glob.glob(logs_glob, root_dir=root_dir)
111115
]
112116

113-
assert data, "no log files found!"
117+
# If not data found, there is nothing to iterate over
118+
if not data:
119+
return [], []
114120

115121
for tup in zip(*data):
116122
read_values = []

0 commit comments

Comments
 (0)