Skip to content

Commit f58b59a

Browse files
small metal benchmark runners (#7456)
Use metal runners to try and reduce run variance.
1 parent f418844 commit f58b59a

3 files changed

Lines changed: 11 additions & 6 deletions

File tree

.github/workflows/nightly-bench.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,4 +72,4 @@ jobs:
7272
matrix:
7373
machine_type:
7474
- id: x86
75-
instance_name: c6id.8xlarge
75+
instance_name: i7i.metal-24xl

.github/workflows/sql-benchmarks.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ on:
99
machine_type:
1010
required: false
1111
type: string
12-
default: c6id.8xlarge
12+
default: i7i.metal-24xl
1313
benchmark_matrix:
1414
required: false
1515
type: string
@@ -270,7 +270,7 @@ jobs:
270270

271271
runs-on: >-
272272
${{ github.repository == 'vortex-data/vortex'
273-
&& format('runs-on={0}/runner=bench-dedicated/tag={1}{2}', github.run_id, matrix.id, (inputs.mode != 'pr' || github.event.pull_request.head.repo.fork == false) && '/extras=s3-cache' || '')
273+
&& format('runs-on={0}/runner=bench-dedicated/instance-type={1}/tag={2}{3}', github.run_id, inputs.machine_type, matrix.id, (inputs.mode != 'pr' || github.event.pull_request.head.repo.fork == false) && '/extras=s3-cache' || '')
274274
|| 'ubuntu-latest' }}
275275
steps:
276276
- uses: runs-on/action@v2

scripts/bench-taskset.sh

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,17 @@ fi
1818

1919

2020
if [[ -z "${BENCH_CPUS:-}" ]]; then
21-
cpu_count="$(nproc)"
22-
BENCH_CPUS="2-$((cpu_count - 1))"
21+
if command -v numactl >/dev/null 2>&1; then
22+
# All CPUs on NUMA node 0, skipping CPUs 0-1 to avoid OS interference
23+
BENCH_CPUS=$(numactl --hardware | awk '/^node 0 cpus:/{sep=""; for(i=4;i<=NF;i++){if($i+0>1){printf "%s%s",sep,$i; sep=","}}}')
24+
else
25+
cpu_count="$(nproc)"
26+
BENCH_CPUS="2-$((cpu_count - 1))"
27+
fi
2328
fi
2429

2530
if command -v numactl >/dev/null 2>&1; then
26-
exec numactl --physcpubind="$BENCH_CPUS" --localalloc "$@"
31+
exec numactl --physcpubind="$BENCH_CPUS" --membind=0 "$@"
2732
fi
2833

2934
exec taskset -c "$BENCH_CPUS" "$@"

0 commit comments

Comments
 (0)