Skip to content

Commit 7f70c2e

Browse files
Spencer Bryngelsonclaude
andcommitted
ci: always nuke build/ on Phoenix to avoid stale ISA mismatches
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent c68d6d6 commit 7f70c2e

1 file changed

Lines changed: 4 additions & 8 deletions

File tree

.github/workflows/common/test.sh

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,10 @@ build_opts="$gpu_opts"
1010

1111
# --- Build (if not pre-built on login node) ---
1212
# Phoenix builds inside SLURM; Frontier pre-builds via build.sh on the login node.
13-
# Phoenix: validate an existing build against this node's CPU ISA.
14-
# A stale build/ from a prior run on a different microarchitecture would SIGILL.
15-
if [ "$job_cluster" = "phoenix" ] && [ -d "build" ]; then
16-
syscheck_bin=$(find build/install -name syscheck -type f 2>/dev/null | head -1)
17-
if [ -n "$syscheck_bin" ] && ! "$syscheck_bin" > /dev/null 2>&1; then
18-
echo "syscheck failed on existing build — nuking build/"
19-
rm -rf build
20-
fi
13+
# Phoenix builds inside SLURM on heterogeneous compute nodes — always start fresh
14+
# to avoid SIGILL from stale binaries compiled on a different microarchitecture.
15+
if [ "$job_cluster" = "phoenix" ]; then
16+
rm -rf build
2117
fi
2218

2319
if [ ! -d "build" ]; then

0 commit comments

Comments
 (0)