执行摘要
修复 NPU CI 配置中变量引用错误
PR 标题和提交信息表明修复 stage-b-test-2-npu-a2 image。从 patch 看,NPU CI 配置中的容器镜像变量引用多了一个 $ 符号,导致变量无法正确展开,CI Extra 测试失败(见 PR body 中 Extra 状态为 ❌)。
值得合并,属于紧急 CI 修复。可关注后续是否有更多类似变量引用错误。
无 review 评论,仅由 sglang-npu-bot 自动批准(APPROVED)。
PR 标题和提交信息表明修复 stage-b-test-2-npu-a2 image。从 patch 看,NPU CI 配置中的容器镜像变量引用多了一个 $ 符号,导致变量无法正确展开,CI Extra 测试失败(见 PR body 中 Extra 状态为 ❌)。
值得合并,属于紧急 CI 修复。可关注后续是否有更多类似变量引用错误。
无 review 评论,仅由 sglang-npu-bot 自动批准(APPROVED)。
在 .github/workflows/pr-test-npu.yml 文件中,将第 155 行的 $${{ needs.set-image-config.outputs.CANN_image_910b }} 改为 ${{ needs.set-image-config.outputs.CANN_image_910b }},即去掉多余的 $ 符号。
| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
.github/workflows/pr-test-npu.yml |
CI 配置 | modified | 2.95 |
分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。
当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。
风险极低:变更只涉及一行 YAML 语法修正,不改变任何逻辑语义,且 Base CI 测试已通过(✅)。
影响范围极小:仅修复 NPU CI 工作流中 Extra 测试套件的容器镜像获取,使其能正确使用预期镜像,避免 CI 失败。
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论