Skip to content

Commit e735cb1

Browse files
committed
ci(ab): merge network test steps
The network latency test only takes 2 minutes, so its a bit silly to spawn up a whole .metal for it. Just run it on the same .metal as the network throughput test. Signed-off-by: Patrick Roy <[email protected]>
1 parent 45fc21b commit e735cb1

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

.buildkite/pipeline_perf.py

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,18 +27,14 @@
2727
"devtool_opts": "-c 1-10 -m 0",
2828
"ab_opts": "--noise-threshold 0.1",
2929
},
30-
"network-latency": {
31-
"label": "📠 Network Latency",
32-
"test_path": "integration_tests/performance/test_network_ab.py::test_network_latency",
30+
"network": {
31+
"label": "📠 Network Latency and Throughput",
32+
"test_path": "integration_tests/performance/test_network_ab.py",
3333
"devtool_opts": "-c 1-10 -m 0",
3434
# Triggers if delta is > 0.01ms (10µs) or default relative threshold (5%)
35+
# only relevant for latency test, throughput test will always be magnitudes above this anyway
3536
"ab_opts": "--absolute-strength 0.010",
3637
},
37-
"network-throughput": {
38-
"label": "📠 Network TCP Throughput",
39-
"test_path": "integration_tests/performance/test_network_ab.py::test_network_tcp_throughput",
40-
"devtool_opts": "-c 1-10 -m 0",
41-
},
4238
"snapshot-latency": {
4339
"label": "📸 Snapshot Latency",
4440
"test_path": "integration_tests/performance/test_snapshot_ab.py",

0 commit comments

Comments
 (0)