Skip to content

Commit 04d5f85

Browse files
committed
test(perf): Drop pytest timeouts
They are remnant of when the performance tests were run without pytest parameterization, meaning all test scenarios were run inside of the same pytest test. This required large timeouts. Now with pytest parameterization, the runtime of each test is at most 30s, which conformtably fits into pytest's default timeout of 6min. Signed-off-by: Patrick Roy <[email protected]>
1 parent d304a01 commit 04d5f85

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

tests/integration_tests/performance/test_block_ab.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ def process_fio_logs(vm, fio_mode, logs_dir, metrics):
138138

139139

140140
@pytest.mark.nonci
141-
@pytest.mark.timeout(RUNTIME_SEC * 1000) # 1.40 hours
142141
@pytest.mark.parametrize("vcpus", [1, 2], ids=["1vcpu", "2vcpu"])
143142
@pytest.mark.parametrize("fio_mode", ["randread", "randwrite"])
144143
@pytest.mark.parametrize("fio_block_size", [4096], ids=["bs4096"])

tests/integration_tests/performance/test_network_ab.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ def network_microvm(request, microvm_factory, guest_kernel, rootfs):
6666

6767

6868
@pytest.mark.nonci
69-
@pytest.mark.timeout(3600)
7069
@pytest.mark.parametrize("network_microvm", [1], indirect=True)
7170
def test_network_latency(
7271
network_microvm, metrics

0 commit comments

Comments
 (0)