ci: fix live server-capture gate (runner lost NO_PROXY) + print server.log on failure - #937
Open
maocheng23 wants to merge 2 commits into
Open
maocheng23 wants to merge 2 commits into
maocheng23 wants to merge 2 commits into
Conversation
The gate only reported the server.log path, which lives in the CI container and is gone after the job. Include the exit code and the last 80 lines so a startup crash is diagnosable from the job log. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The self-hosted runner still exports HTTP_PROXY/HTTPS_PROXY into the job container but stopped exporting NO_PROXY (present on 09-30, absent since). sglang's startup warmup POSTs to 127.0.0.1, went through the proxy, got a 502 and the server killed itself, failing the live server-capture gate on every PR. Set NO_PROXY for loopback at the job level. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
maocheng23
force-pushed
the
ci/capture-gate-dump-server-log
branch
from
October 6, 2026 06:14
d55ff90 to
2a63211
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
The live server-capture gate in
PR Testfails on every PR since 09-30, for example #934 (twice) and this PR's first run:Root cause: the self-hosted runner still exports
HTTP_PROXY/HTTPS_PROXY=http://172.17.0.1:7890into the job container, butNO_PROXY/no_proxyare no longer exported. They were present in the last green run (09-30,NO_PROXY=localhost,127.0.0.1,::1,...) and are absent in every run since. sglang's startup warmup POSTs to127.0.0.1, so the request goes to the proxy, which returns 502, and the server kills itself.Changes
NO_PROXY=localhost,127.0.0.1,::1at theunit-testjob level, so loopback traffic bypasses the proxy regardless of the runner's environment.server.login the error. That's how this cause was found; before, the job log only gave a path inside the discarded container.Runner admins may also want to restore
NO_PROXYin the runner's own environment, since other loopback traffic in the job is affected too.🤖 Generated with Claude Code