Skip to content
Closed
Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
cddd605
Merge benchmarks into test.yml for parallel CI execution
sbryngelson Feb 9, 2026
5116e50
Remove cancel-on-bench-failure to avoid killing tests on bench flakes
sbryngelson Feb 9, 2026
5b75291
Consolidate Frontier SLURM jobs into multi-node allocations
sbryngelson Feb 9, 2026
8c14b05
Fix cp-into-self error in consolidated Frontier test script
sbryngelson Feb 9, 2026
50e43f5
Fix review issues: rocm-smi gating, subshell env, job ID parsing
sbryngelson Feb 9, 2026
eebd753
Move concurrency from workflow-level to per-job
sbryngelson Feb 9, 2026
8d7f492
Parallelize Frontier builds on login node
sbryngelson Feb 10, 2026
d852af3
Restore workflow-level concurrency for full run cancellation
sbryngelson Feb 10, 2026
1627311
Add build progress heartbeat and collapsible GHA log groups
sbryngelson Feb 10, 2026
d98e894
Fix heartbeat kill guard and hardcoded node count in log message
sbryngelson Feb 10, 2026
b5f6c04
Clean up stale dirs before creating source copies
sbryngelson Feb 10, 2026
6e173e4
Switch Phoenix GPU jobs to H200 nodes for faster scheduling
sbryngelson Feb 10, 2026
553ff35
Harden Frontier CI orchestration scripts
sbryngelson Feb 10, 2026
e5ce527
Limit parallel builds to 2 concurrent on login node
sbryngelson Feb 11, 2026
de1b6d5
Set SETUPTOOLS_SCM_PRETEND_VERSION for hardlink source copies
sbryngelson Feb 11, 2026
1347a54
Merge branch 'master' into readme
sbryngelson Feb 11, 2026
0a5a0f6
Add build heartbeat during throttled build phase
sbryngelson Feb 12, 2026
2575eb7
Merge branch 'master' into readme
sbryngelson Feb 12, 2026
e2cb418
Fix bash segfault in monitor_slurm_job.sh from fractional read timeout
sbryngelson Feb 12, 2026
580d1f2
Use real copies instead of hardlinks and increase parallel builds to 3
sbryngelson Feb 12, 2026
9d509fb
Remove NODE_OPTIONS from CI workflow
sbryngelson Feb 12, 2026
fe79a51
Add workspace pre-clean to avoid stale NFS handles on self-hosted run…
sbryngelson Feb 12, 2026
0006d6f
Isolate build temp dirs to prevent Cray compiler conflicts
sbryngelson Feb 13, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/file-filter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ yml: &yml
- '.github/workflows/phoenix/**'
- '.github/workflows/frontier/**'
- '.github/workflows/frontier_amd/**'
- '.github/workflows/bench.yml'
- '.github/workflows/test.yml'
- '.github/workflows/formatting.yml'

Expand Down
34 changes: 34 additions & 0 deletions .github/scripts/frontier_bench_config.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#!/bin/bash
# Run a single benchmark on a Frontier compute node (build already done on login node).
# Usage: frontier_bench_config.sh <cluster> <device> <interface>
# Runs inside a SLURM allocation on an ssh'd compute node.

set -e
set -x

cluster=$1; device=$2; interface=$3

flag="f"; [ "$cluster" = "frontier_amd" ] && flag="famd"
mode="g"; [ "$device" = "cpu" ] && mode="c"

. ./mfc.sh load -c "$flag" -m "$mode"

# Benchmark
job_slug="bench-${device}-${interface}"
n_ranks=12
device_opts=""
if [ "$device" = "gpu" ]; then
gpus=$(rocm-smi --showid | awk '{print $1}' | grep -Eo '[0-9]+' | uniq | tr '\n' ' ')
n_ranks=$(echo "$gpus" | wc -w)
gpu_ids=$(echo "$gpus" | tr ' ' '\n' | tr '\n' ' ' | sed 's/ $//')
device_opts="--gpu"
[ "$interface" = "acc" ] && device_opts+=" acc"
[ "$interface" = "omp" ] && device_opts+=" mp"
device_opts+=" -g $gpu_ids"
fi
Comment thread
sbryngelson marked this conversation as resolved.
Comment thread
sbryngelson marked this conversation as resolved.

if [ "$device" = "gpu" ]; then
./mfc.sh bench --mem 12 -j $n_ranks -o "$job_slug.yaml" -- -c "$cluster" $device_opts -n $n_ranks
else
./mfc.sh bench --mem 1 -j $(nproc) -o "$job_slug.yaml" -- -c "$cluster" $device_opts -n $n_ranks
Comment thread
sbryngelson marked this conversation as resolved.
fi
Comment thread
sbryngelson marked this conversation as resolved.
Comment thread
sbryngelson marked this conversation as resolved.
35 changes: 35 additions & 0 deletions .github/scripts/frontier_bench_post.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
#!/bin/bash
# Post-process all Frontier benchmark results after the SLURM job completes.
# Runs bench_diff for each config, comparing master vs PR YAML outputs.

set -euo pipefail

# Benchmark configs: cluster device interface flag
bench_configs=(
"frontier:gpu:acc:f"
"frontier:gpu:omp:f"
"frontier_amd:gpu:omp:famd"
)

for cfg in "${bench_configs[@]}"; do
IFS=':' read -r cluster device interface flag <<< "$cfg"
pr_yaml="pr-${cluster}-${device}-${interface}/bench-${device}-${interface}.yaml"
master_yaml="master-${cluster}-${device}-${interface}/bench-${device}-${interface}.yaml"

echo "=========================================="
echo "bench_diff: $cluster $device $interface"
echo " PR: $pr_yaml"
echo " Master: $master_yaml"
echo "=========================================="

if [ ! -f "$pr_yaml" ]; then
echo "ERROR: PR YAML not found: $pr_yaml"
exit 1
fi
if [ ! -f "$master_yaml" ]; then
echo "ERROR: Master YAML not found: $master_yaml"
exit 1
fi

(cd pr && . ./mfc.sh load -c "$flag" -m g && ./mfc.sh bench_diff "../$master_yaml" "../$pr_yaml")
done
34 changes: 34 additions & 0 deletions .github/scripts/frontier_test_config.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
#!/bin/bash
# Run a single test on a Frontier compute node (build already done on login node).
# Usage: frontier_test_config.sh <cluster> <device> <interface>
# Runs inside a SLURM allocation on an ssh'd compute node.

set -e
set -x

cluster=$1; device=$2; interface=$3

flag="f"; [ "$cluster" = "frontier_amd" ] && flag="famd"
mode="g"; [ "$device" = "cpu" ] && mode="c"

. ./mfc.sh load -c "$flag" -m "$mode"

# Device options
device_opts=""
if [ "$device" = "gpu" ]; then
device_opts="--gpu"
[ "$interface" = "acc" ] && device_opts+=" acc"
[ "$interface" = "omp" ] && device_opts+=" mp"
fi
Comment thread
sbryngelson marked this conversation as resolved.

rdma=""
[ "$cluster" = "frontier" ] && [ "$device" = "gpu" ] && rdma="--rdma-mpi"

# Test
if [ "$device" = "gpu" ]; then
gpus=$(rocm-smi --showid | awk '{print $1}' | grep -Eo '[0-9]+' | uniq | tr '\n' ' ')
ngpus=$(echo "$gpus" | wc -w)
./mfc.sh test -v -a $rdma --max-attempts 3 -j $ngpus $device_opts -- -c "$cluster"
else
./mfc.sh test -v -a --max-attempts 3 -j 32 --no-gpu -- -c "$cluster"
fi
220 changes: 220 additions & 0 deletions .github/scripts/run_frontier_all_benchmarks.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,220 @@
#!/bin/bash
# Orchestrate all Frontier benchmark configs in one multi-node SLURM allocation.
# 1. Builds all configs on the login node (PR and master, in parallel)
# 2. Submits a single 6-node SLURM job running benchmarks in parallel via ssh

set -euo pipefail

# Ignore SIGHUP to survive login node session drops
trap '' HUP

SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"

# Benchmark configs: version cluster device interface
# 6 total: 3 configs x 2 versions (PR + master)
configs=(
"pr frontier gpu acc"
"pr frontier gpu omp"
"pr frontier_amd gpu omp"
"master frontier gpu acc"
"master frontier gpu omp"
"master frontier_amd gpu omp"
)
num_nodes=${#configs[@]}

echo "=========================================="
echo "Frontier consolidated benchmarks: $num_nodes configs on $num_nodes nodes"
echo "=========================================="

# --- Phase 1: Create per-config source copies ---
for cfg in "${configs[@]}"; do
read -r version cluster device interface <<< "$cfg"
dir="${version}-${cluster}-${device}-${interface}"
echo "Creating source copy: $dir from $version/"
cp -al "$version" "$dir" 2>/dev/null || cp -r "$version" "$dir"
done
Comment thread
sbryngelson marked this conversation as resolved.

# --- Phase 2: Build all configs on login node in parallel ---
echo ""
echo "=========================================="
echo "Starting parallel builds (${num_nodes} configs)..."
echo "=========================================="

build_pids=()
for cfg in "${configs[@]}"; do
read -r version cluster device interface <<< "$cfg"
dir="${version}-${cluster}-${device}-${interface}"
log="build-${version}-${cluster}-${device}-${interface}.log"
echo " Starting: $version $cluster $device $interface"
(
cd "$dir"
bash .github/workflows/${cluster}/build.sh "$device" "$interface" bench
) > "$log" 2>&1 &
build_pids+=($!)
done

# Periodic heartbeat while builds run
(
while true; do
sleep 120
alive=0
for pid in "${build_pids[@]}"; do
kill -0 "$pid" 2>/dev/null && alive=$((alive + 1))
done
[ "$alive" -eq 0 ] && break
echo "--- Build heartbeat ($(date +%H:%M:%S)): $alive/${#build_pids[@]} running ---"
for i in "${!configs[@]}"; do
read -r version cluster device interface <<< "${configs[$i]}"
log="build-${version}-${cluster}-${device}-${interface}.log"
if kill -0 "${build_pids[$i]}" 2>/dev/null; then
size=$(stat -c%s "$log" 2>/dev/null || echo 0)
last=$(tail -n 1 "$log" 2>/dev/null | head -c 120 || echo "")
echo " $version $cluster $device $interface: running (${size} bytes) $last"
fi
done
done
) &
heartbeat_pid=$!

# Wait for all builds and report results
build_failed=0
build_exits=()
for i in "${!build_pids[@]}"; do
read -r version cluster device interface <<< "${configs[$i]}"
if wait "${build_pids[$i]}"; then
build_exits+=(0)
else
code=$?
build_exits+=($code)
build_failed=1
Comment thread
sbryngelson marked this conversation as resolved.
fi
done

# Stop heartbeat
kill "$heartbeat_pid" 2>/dev/null; wait "$heartbeat_pid" 2>/dev/null || true
Comment thread
cubic-dev-ai[bot] marked this conversation as resolved.
Outdated

# Print build logs: passed builds collapsed, failed builds in full
for i in "${!configs[@]}"; do
read -r version cluster device interface <<< "${configs[$i]}"
log="build-${version}-${cluster}-${device}-${interface}.log"
if [ "${build_exits[$i]}" -eq 0 ]; then
echo "::group::Build PASSED: $version $cluster $device $interface"
cat "$log"
echo "::endgroup::"
else
echo "=========================================="
echo "Build FAILED: $version $cluster $device $interface (exit ${build_exits[$i]})"
echo "=========================================="
cat "$log"
fi
done

# Abort on failure
if [ "$build_failed" -ne 0 ]; then
echo ""
echo "=========================================="
echo "Build failures detected — see logs above."
echo "=========================================="
exit 1
fi

echo ""
echo "=========================================="
echo "All builds complete. Submitting ${num_nodes}-node SLURM job..."
echo "=========================================="

# --- Phase 3: Submit one sbatch job with N nodes ---
output_file="bench-frontier-all.out"

submit_output=$(sbatch <<'OUTER'
#!/bin/bash
#SBATCH -J MFC-frontier-all-bench
#SBATCH -A ENG160
#SBATCH -N 6
#SBATCH -t 05:59:00
#SBATCH -obench-frontier-all.out
#SBATCH -p extended

set -x

cd "$SLURM_SUBMIT_DIR"
echo "Running in $(pwd)"
echo "Allocated nodes: $SLURM_NODELIST"

# Get list of individual node hostnames
mapfile -t nodes < <(scontrol show hostnames "$SLURM_NODELIST")
echo "Nodes: ${nodes[*]}"

# Config table (must match the outer script)
configs=(
"pr frontier gpu acc"
"pr frontier gpu omp"
"pr frontier_amd gpu omp"
"master frontier gpu acc"
"master frontier gpu omp"
"master frontier_amd gpu omp"
)

pids=()

for i in "${!configs[@]}"; do
read -r version cluster device interface <<< "${configs[$i]}"
node="${nodes[$i]}"
dir="${version}-${cluster}-${device}-${interface}"
outfile="${dir}/bench-${device}-${interface}.out"

echo "[$node] Starting bench: $version $cluster $device $interface in $dir"

ssh -q -o StrictHostKeyChecking=no "$node" \
"cd $SLURM_SUBMIT_DIR/$dir && bash .github/scripts/frontier_bench_config.sh $cluster $device $interface" \
> "$outfile" 2>&1 &
pids+=($!)
done
Comment thread
sbryngelson marked this conversation as resolved.
Outdated

echo "All bench configs launched, waiting for completion..."

# Wait for all and collect exit codes
overall_exit=0
for i in "${!pids[@]}"; do
read -r version cluster device interface <<< "${configs[$i]}"
pid=${pids[$i]}
if wait "$pid"; then
echo "PASSED: $version $cluster $device $interface (PID $pid)"
else
code=$?
echo "FAILED: $version $cluster $device $interface (PID $pid, exit code $code)"
overall_exit=1
fi
done

# Print summary
echo ""
echo "=========================================="
echo "Benchmark summary:"
for cfg in "${configs[@]}"; do
read -r version cluster device interface <<< "$cfg"
dir="${version}-${cluster}-${device}-${interface}"
yaml="${dir}/bench-${device}-${interface}.yaml"
if [ -f "$yaml" ]; then
echo " $version $cluster $device $interface: OK ($(stat -c%s "$yaml" 2>/dev/null) bytes)"
else
echo " $version $cluster $device $interface: MISSING YAML"
fi
done
echo "=========================================="

exit $overall_exit
OUTER
)

job_id=$(echo "$submit_output" | awk '/Submitted batch job/ {print $4}')
if [ -z "$job_id" ]; then
echo "ERROR: Failed to submit job. sbatch output:"
echo "$submit_output"
exit 1
fi

echo "Submitted batch job $job_id ($num_nodes nodes)"

# Monitor the job
bash "$SCRIPT_DIR/monitor_slurm_job.sh" "$job_id" "$output_file"
Loading
Loading