Prhub

#24974 Register SWA unit tests under unit/mem_cache

原始 PR 作者 ispobock 合并时间 2026-05-11 20:21 文件变更 2 提交数 1 评论 2 代码增减 +6 / -0

执行摘要

将 SWA 单元测试从手动目录迁移到 CI 注册目录

PR body 明确说明:将两个 SWA 单元测试从 test/manual/dsv4/ 提升到 test/registered/unit/mem_cache/,以便在 CI 中运行。

可直接合并。这是一个典型的测试基建改进,值得关注的是这些测试覆盖了 SWA 内存池的关键路径(页分配扩展和锁释放),对维护稳定性有积极作用。

讨论亮点

只有 gemini-code-assist[bot] 自动评论表示无反馈。无人工 review 讨论。

实现拆解

  1. 将 test/manual/dsv4/test_swa_alloc_extend_page_estimation.py 重命名为 test/registered/unit/mem_cache/test_swa_alloc_extend_page_estimation.py。
  2. 在新文件中添加 from sglang.test.ci.ci_register import register_cpu_ci 导入,并调用 register_cpu_ci(est_time=2, suite="stage-a-test-cpu")。
  3. 将 test/manual/dsv4/test_swa_lock_release_lifecycle.py 重命名为 test/registered/unit/mem_cache/test_swa_lock_release_lifecycle.py。
  4. 在新文件中添加 from sglang.test.ci.ci_register import register_cuda_ci 导入,并调用 register_cuda_ci(est_time=12, suite="stage-b-test-1-gpu-small")。
  5. 两个测试文件内容本身无其他变更。
文件 模块 状态 重要度
test/registered/unit/mem_cache/test_swa_alloc_extend_page_estimation.py 测试 renamed 3.92
test/registered/unit/mem_cache/test_swa_lock_release_lifecycle.py 测试 renamed 3.92

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

评论区精华

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

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

风险与影响

风险极低。变更仅涉及测试文件重命名和添加 CI 注册装饰器,不影响任何生产代码。测试可能在 CI 环境中执行时间较长(est_time=12),但已通过注册参数告知调度器。

影响范围仅限于 CI 流程:两个 SWA 单元测试(SWA 页分配扩展估算回归测试和 SWA 锁释放生命周期回归测试)将定期在 CI 中运行,提高对 SWA 内存管理模块的回归覆盖。开发者在本地运行测试时仍可手动触发。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论