Prhub

#29596 Fix CI caused by https://github.com/sgl-project/sglang/pull/29576

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

执行摘要

修复 DSA indexer 测试因 device 参数错误导致的 CI 失败

29576 修复了 DSA indexer 的内存问题,但其测试用例 test_dsa_indexer.py 中的 _create_indexer 方法在创建 Indexer 实例时没有指定 device,导致 Indexer 在 CPU 上创建,CI 报错。本 PR 快速修复此问题,使测试通过(11 passed)。

该 PR 是纯粹的 CI 修复,简单直接。值得关注的是其揭示了测试代码应尽量贴近真实运行环境(如 device 参数),建议后续测试编写时注意此类细节。

讨论亮点

无 review 讨论。PR author b8zhong 提交后,hnyls2002 触发 rerun-test,测试通过后 PR 被合并。

实现拆解

test/registered/kernels/test_dsa_indexer.py_create_indexer 方法中,将 indexer = Indexer(**params) 包裹在 with torch.device(self.device): 上下文内,确保 Indexer 直接在 CUDA 设备上初始化,与后续 indexer.to(device=self.device) 保持一致。

文件 模块 状态 重要度
test/registered/kernels/test_dsa_indexer.py 测试 modified 3.99

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

评论区精华

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

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

风险与影响

变更仅影响测试代码,风险极低。但需注意,之前 PR #29576 的修复效果仍需通过该测试验证。

仅影响 DSA indexer 测试的 CI 流程,用户无感知。确保持续集成通过,防止回归。

测试配套变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论