From 9eb4da308d6ff42abf4a51427c69160e0fdc29d2 Mon Sep 17 00:00:00 2001 From: Patrick Roy Date: Wed, 8 Jan 2025 16:40:59 +0000 Subject: [PATCH 1/2] test: unpin m6i/5.10 AMI for perf tests With C-states renabled by f610cae59a5d ("fix: stop messing with C-states in performance tests"), the m6i volatility will be gone. Signed-off-by: Patrick Roy --- .buildkite/pipeline_perf.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/.buildkite/pipeline_perf.py b/.buildkite/pipeline_perf.py index e8169bfb2cd..a94c13b8af0 100755 --- a/.buildkite/pipeline_perf.py +++ b/.buildkite/pipeline_perf.py @@ -121,10 +121,7 @@ # } # will pin steps running on instances "m6i.metal" with kernel version tagged "linux_6.1" # to a new kernel version tagged "linux_6.1-pinned" -pins = { - # TODO: Unpin when performance instability on m6i/5.10 has gone. - "linux_5.10-pinned": {"instance": "m6i.metal", "kv": "linux_5.10"}, -} +pins = {} def apply_pins(steps): From 4e6011cefcf6e7fbd809ca3a81123eb9d394addf Mon Sep 17 00:00:00 2001 From: Jack Thomson Date: Wed, 11 Dec 2024 16:57:28 +0000 Subject: [PATCH 2/2] Use 0ms delay on latency tests Using the larger delay appears to have negative side affects on AMD Signed-off-by: Jack Thomson --- tests/integration_tests/performance/test_network_ab.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/integration_tests/performance/test_network_ab.py b/tests/integration_tests/performance/test_network_ab.py index 60c4f17361e..3a50d864544 100644 --- a/tests/integration_tests/performance/test_network_ab.py +++ b/tests/integration_tests/performance/test_network_ab.py @@ -62,7 +62,7 @@ def test_network_latency(network_microvm, metrics): rounds = 15 request_per_round = 30 - delay = 0.2 + delay = 0.0 metrics.set_dimensions( {