Skip to content

Commit c34792f

Browse files
joe4devclaude
andcommitted
fix(ls-api): fix smoke test on Apple Silicon, improve container debug output
Two fixes: 1. Add --platform linux/amd64 to docker run so Docker always pulls the x86_64 image. On Apple Silicon, Docker otherwise selects the arm64 image, causing an exec format error when mounting the linux/amd64 RIE binary -- the container exits immediately without sending any callbacks. 2. Drop --rm and do explicit docker rm in cleanup instead. With --rm the container and its logs are deleted on exit before wait_for_log can retrieve them, making failures invisible. Without --rm, docker logs and docker inspect work correctly even after the container has stopped. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 785d93e commit c34792f

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

cmd/ls-api/smoke-test.sh

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,10 @@ CID_FILE=$(mktemp -t rie-smoke.XXXXXX)
1818
cleanup() {
1919
local cid
2020
cid=$(cat "$CID_FILE" 2>/dev/null || true)
21-
[ -n "$cid" ] && docker stop "$cid" 2>/dev/null || true
21+
if [ -n "$cid" ]; then
22+
docker stop "$cid" 2>/dev/null || true
23+
docker rm -f "$cid" 2>/dev/null || true
24+
fi
2225
[ -n "${MOCK_PID:-}" ] && kill "$MOCK_PID" 2>/dev/null || true
2326
rm -f "$LOG_FILE" "$CID_FILE"
2427
}
@@ -55,7 +58,8 @@ for i in $(seq 1 10); do nc -z localhost $MOCK_PORT 2>/dev/null && break || slee
5558
echo ">>> Starting RIE in Docker"
5659

5760
docker_opts=(
58-
--rm --detach
61+
--detach
62+
--platform linux/amd64
5963
--add-host=host.docker.internal:host-gateway
6064
-p "$INTEROP_PORT:$INTEROP_PORT"
6165
-v "$RIE_BINARY:/var/rapid/init:ro"

0 commit comments

Comments
 (0)