执行摘要
测试中繁忙内存检查降级为安静模式
PR body 明确指出:"Same per-step leak assertion, far less CI log spam." 目的是减少 CI 日志噪音,优化调试体验。
可直接合入,变更简单且意图明确。建议在日后 CI 调试时,若怀疑内存泄漏相关故障,可临时切换回 level 2 获取详细日志。
无 review 讨论。
PR body 明确指出:"Same per-step leak assertion, far less CI log spam." 目的是减少 CI 日志噪音,优化调试体验。
可直接合入,变更简单且意图明确。建议在日后 CI 调试时,若怀疑内存泄漏相关故障,可临时切换回 level 2 获取详细日志。
无 review 讨论。
test/registered/scheduler/test_mixed_chunked_prefill.py 中,将 setUpClass 里的环境变量覆盖值从 2 改为 1。python/sglang/test/server_fixtures/streaming_session_fixture.py 中,更新了类文档字符串和 setUpClass 中的环境变量覆盖值,同样从 2 改为 1。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
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 链接,后续同步到相关引用后会出现在这里。
参与讨论