Prhub

#27303 Use level-1 (quiet) busy memory check in chunked-prefill and streaming tests

原始 PR 作者 hnyls2002 合并时间 2026-06-05 05:04 文件变更 2 提交数 1 评论 1 代码增减 +3 / -3

执行摘要

测试中繁忙内存检查降级为安静模式

PR body 明确指出:"Same per-step leak assertion, far less CI log spam." 目的是减少 CI 日志噪音,优化调试体验。

可直接合入,变更简单且意图明确。建议在日后 CI 调试时,若怀疑内存泄漏相关故障,可临时切换回 level 2 获取详细日志。

讨论亮点

无 review 讨论。

实现拆解

  1. 修改 chunked-prefill 测试:在 test/registered/scheduler/test_mixed_chunked_prefill.py 中,将 setUpClass 里的环境变量覆盖值从 2 改为 1
  2. 修改 streaming-session 测试 fixture:在 python/sglang/test/server_fixtures/streaming_session_fixture.py 中,更新了类文档字符串和 setUpClass 中的环境变量覆盖值,同样从 2 改为 1
  3. 未修改其他逻辑:两个测试的启动参数、断言逻辑和 teardown 保持不变,仅切换了内存检查的日志详细级别。
文件 模块 状态 重要度
test/registered/scheduler/test_mixed_chunked_prefill.py 测试 modified 3.46
python/sglang/test/server_fixtures/streaming_session_fixture.py 测试 modified 3.43

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低。level 1 仍会按 step 检查内存泄漏,仅在泄漏时输出日志,不会影响泄漏检测的正确性。唯一可能的变化是调试时若没有泄漏则看不到池统计信息,但这正是预期的安静行为。

影响范围仅限于 CI 日志输出:chunked-prefill 和 streaming-session 测试的日志将大幅减少,便于在大量测试输出中快速定位真正的问题。对用户功能无影响。

测试变更

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论