Prhub

#21547 [CI] Register missing jit_kernel test files

原始 PR 作者 BBuf 合并时间 2026-03-27 19:39 文件变更 4 提交数 1 评论 2 代码增减 +14 / -0

执行摘要

为 jit_kernel 的 cast 和 fused_qknorm_rope 测试与基准添加 CUDA CI 注册条目。

PR body 中明确说明:'Add CUDA CI registry entries for the unregistered cast and fused_qknorm_rope tests and benchmarks so run_suite.py can collect them again.',即添加 CUDA CI 注册条目以使未注册的测试和基准能被 CI 系统收集,避免遗漏执行。

对于技术管理者,此 PR 无需精读,但值得关注以确保 CI 测试覆盖完整,可作为基础设施变更的参考。工程师可快速浏览以了解 CI 注册机制和 est_time 设置方法,无重大设计决策需要学习。

讨论亮点

review 中无实质讨论,仅有的评论来自 gemini-code-assist[bot],表示无反馈提供。因此,未产生争议、设计权衡或未解决疑虑。

实现拆解

实现涉及四个文件:两个基准文件(bench_cast.py 和 bench_fused_qknorm_rope.py)和两个单元测试文件(test_cast.py 和 test_fused_qknorm_rope.py)。在每个文件中,添加了 from sglang.test.ci.ci_register import register_cuda_ci 导入和 register_cuda_ci 调用,指定了 est_time 和 suite 参数。例如,test_cast.py 设置了 est_time=15 for stage-b-kernel-unit-1-gpu-large 和 est_time=120 for nightly-kernel-1-gpu(nightly=True)。

文件 模块 状态 重要度
python/sglang/jit_kernel/benchmark/bench_cast.py jit_kernel benchmark modified 3.0
python/sglang/jit_kernel/benchmark/bench_fused_qknorm_rope.py jit_kernel benchmark modified 3.0
python/sglang/jit_kernel/tests/test_cast.py jit_kernel tests modified 3.0
python/sglang/jit_kernel/tests/test_fused_qknorm_rope.py jit_kernel tests modified 3.0

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

评论区精华

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

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

风险与影响

风险较低:变更仅添加 CI 注册调用,不修改核心代码逻辑,无回归、性能、安全或兼容性问题。潜在风险包括 est_time 值设置可能不准确,导致 CI 超时或资源浪费,但基于 H200 验证的值应较可靠。此外,如果 register_cuda_ci 函数有未预料的行为,可能影响 CI 收集,但鉴于其基础设施性质,影响有限。

影响范围限于 CI 系统:确保 cast 和 fused_qknorm_rope 的测试与基准被正确收集和运行,提高测试覆盖率和 CI 可靠性。对用户、系统核心功能或团队日常工作无直接影响。影响程度为低,属于常规基础设施维护。

配置变更 低风险 无核心代码变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论