Skip to content

Commit d3e09f8

Browse files
Sudan Landgewearyzen
authored andcommitted
fix: new intermittent ci issue
While running AB perf tests with new kvm_metrics enabled a new sock intermittent ci issues was found and ping_latency showed a regression. vsock iperf test fails intermittently with error: `/tmp/iperf3-vsock: Text file busy` ping_latency regression was not consistent and fixed on retry. Since the only variable here is collecting metrics adjusting the point of collection for final metrics for performance tests fixes both the issues. Signed-off-by: Sudan Landge <[email protected]>
1 parent 856411c commit d3e09f8

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

tests/integration_tests/performance/test_network_ab.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,10 +96,10 @@ def test_network_latency(network_microvm, metrics, iteration):
9696
assert rc == 0, stderr
9797

9898
samples.extend(consume_ping_output(ping_output))
99+
fcmetrics.stop()
99100

100101
for sample in samples:
101102
metrics.put_metric("ping_latency", sample, "Milliseconds")
102-
fcmetrics.stop()
103103

104104

105105
class TcpIPerf3Test(IPerf3Test):

tests/integration_tests/performance/test_vsock_ab.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,5 +110,5 @@ def test_vsock_throughput(
110110
test = VsockIPerf3Test(vm, mode, payload_length)
111111
data = test.run_test(vm.vcpus_count + 2)
112112

113-
emit_iperf3_metrics(metrics, data, VsockIPerf3Test.WARMUP_SEC)
114113
fcmetrics.stop()
114+
emit_iperf3_metrics(metrics, data, VsockIPerf3Test.WARMUP_SEC)

0 commit comments

Comments
 (0)