Skip to content

Commit e864803

Browse files
sbryngelsonclaude
andcommitted
ci: always rebuild on Phoenix even if rm -rf build leaves partial directory
If stale NFS handles prevent full deletion of build/, the old 'if [ ! -d build ]' guard would skip the rebuild entirely, leaving stale binaries from a different compute node that could cause SIGILL. Force a rebuild on Phoenix unconditionally. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7f467b1 commit e864803

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/common/bench.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ if [ "$job_cluster" = "phoenix" ]; then
2929
rm -rf build 2>/dev/null || true
3030
fi
3131

32-
if [ ! -d "build" ]; then
32+
if [ "$job_cluster" = "phoenix" ] || [ ! -d "build" ]; then
3333
source .github/scripts/retry-build.sh
3434
retry_build ./mfc.sh build -j $n_jobs $build_opts || exit 1
3535
fi

.github/workflows/common/test.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,9 @@ if [ "$job_cluster" = "phoenix" ]; then
1717
rm -rf build 2>/dev/null || true
1818
fi
1919

20-
if [ ! -d "build" ]; then
20+
# Phoenix must always rebuild (heterogeneous compute nodes → ISA mismatch risk),
21+
# even if rm above left a partial build/ directory behind.
22+
if [ "$job_cluster" = "phoenix" ] || [ ! -d "build" ]; then
2123
source .github/scripts/retry-build.sh
2224

2325
# Phoenix: smoke-test the syscheck binary to catch architecture mismatches

0 commit comments

Comments
 (0)