Skip to content

Commit ef43d4f

Browse files
committed
test: don't compare the process startup to a threshold
We will keep the metric but not fail the test, since we already fail if the boottime regresses. Signed-off-by: Pablo Barbáchano <[email protected]>
1 parent 038c0e0 commit ef43d4f

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

tests/integration_tests/performance/test_process_startup_time.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
import json
88
import os
9-
import platform
109
import time
1110

1211
import pytest
@@ -15,10 +14,6 @@
1514
from framework.properties import global_props
1615
from host_tools.cargo_build import run_seccompiler_bin
1716

18-
# The maximum acceptable startup time in CPU us.
19-
MAX_STARTUP_TIME_CPU_US = {"x86_64": 5500, "aarch64": 4000}
20-
MAX_STARTUP_TIME = MAX_STARTUP_TIME_CPU_US[platform.machine()]
21-
2217

2318
@pytest.fixture
2419
def startup_time(metrics, record_property):
@@ -34,7 +29,6 @@ def startup_time(metrics, record_property):
3429
def record_startup_time(startup_time):
3530
metrics.put_metric("startup_time", startup_time, unit="Microseconds")
3631
record_property("startup_time_μs", startup_time)
37-
record_property("startup_max_threshold_μs", MAX_STARTUP_TIME)
3832

3933
return record_startup_time
4034

@@ -97,7 +91,6 @@ def _test_startup_time(microvm):
9791
)
9892

9993
assert cpu_startup_time_us > 0
100-
assert cpu_startup_time_us <= MAX_STARTUP_TIME
10194
return cpu_startup_time_us
10295

10396

0 commit comments

Comments
 (0)