Skip to content

Commit 58249c2

Browse files
fix(ci): use bash variable for PID in ssd-draft-memory-guard
GitHub Actions output contexts (${{ steps.X.outputs.Y }}) are not populated until the step finishes. Trying to use it inside the same step resulted in an empty string being passed to 'kill -0', causing the health check to instantly abort the test runner. Switched to standard bash '0' capturing.
1 parent c8b236d commit 58249c2

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -437,11 +437,12 @@ jobs:
437437
--port 15473 \
438438
--max-tokens 64 \
439439
> /tmp/ssd_draft_guard.log 2>&1 &
440-
echo "server_pid=$!" >> $GITHUB_OUTPUT
440+
PID=$!
441+
echo "server_pid=$PID" >> $GITHUB_OUTPUT
441442
442443
echo "Waiting for server (up to 300s)..."
443444
for i in $(seq 1 300); do
444-
if ! kill -0 ${{ steps.server.outputs.server_pid }} 2>/dev/null; then
445+
if ! kill -0 $PID 2>/dev/null; then
445446
echo "Server died early:"
446447
cat /tmp/ssd_draft_guard.log
447448
exit 1

0 commit comments

Comments
 (0)