sgl-project/sglang · 作者 iforgetmyname · 合并时间 2026-03-23 14:36
根据 PR 标题和提交信息，动机是修复 NPU CI 失败，具体问题为 git 报告 dubious ownership（可疑所有权），这可能是由于 Git 2.35+ 版本的安全增强导致的。
此 PR 变更简单，建议快速浏览以了解 CI 配置修复方法，无需深入代码分析。对于负责 CI 维护的工程师，可参考此修复处理类似 git 安全检查问题。
review 评论为空，表明此变更没有经过讨论，可能是简单修复直接合并。
实现方案单一：修改文件 '.github/workflows/pr-test-npu.yml'，在六个不同的 job 中添加一个名为 'Mark repository safe' 的步骤。该步骤运行命令 git config --system --add safe.directory ${GITHUB_WORKSPACE}，以配置 git 信任工作区目录。
| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
.github/workflows/pr-test-npu.yml |
CI 工作流 | modified | 2.0 |
分析完成后，这里会展示 LLM 生成的相对完整源码片段和详细注释。
当前评论区没有形成足够清晰的争议点或结论，后续有更多讨论时会体现在这里。
风险较低：变更仅涉及 CI 配置，不修改代码逻辑。潜在风险包括 git 配置可能影响其他 CI 环境，但鉴于针对特定 NPU 工作流，影响有限。需确保命令正确执行，避免因权限问题导致配置失败。
影响范围：仅限于 NPU CI 工作流，确保 CI 测试能正常运行。影响程度：修复了 CI 失败问题，对开发者体验和代码集成流程有正面影响，但不直接影响用户功能或系统性能。
当前没有检测到明确关联的 Issue 链接，后续同步到相关引用后会出现在这里。
此 PR 修复了 NPU CI 工作流中因 git 安全检查失败的问题，通过在多个 job 中添加 git safe.directory 配置，确保 CI 能正常执行，变更简单且影响有限。
动机是修复 NPU CI 失败，具体问题为 git 报告 dubious ownership（可疑所有权）。根据 PR 标题和提交信息“fix npu ci failure”，这可能是由于 Git 2.35+ 版本的安全增强导致，在 CI 环境中当工作区目录所有权可疑时引发错误。
实现方案仅修改一个文件：.github/workflows/pr-test-npu.yml。在六个不同的 job 中（如 test_npu_on_cpu、test_npu_on_npu 等）均添加了以下步骤：
- name: Mark repository safe
run: |
git config --system --add safe.directory ${GITHUB_WORKSPACE}
此命令将工作区目录标记为安全，避免 git 安全检查报错。变更纯属配置层面，不涉及代码逻辑调整。
review 评论为空，表明此变更未经过讨论，可能是作者直接合并以快速修复 CI 问题。
风险较低：变更仅针对 CI 配置，无核心代码改动，但需确保 git 命令在所有 NPU CI 环境中正确执行，避免因系统权限或配置冲突导致新问题。影响范围仅限于 NPU CI 工作流，修复后能提升开发者的 CI 通过率，对系统功能无直接影响。
与历史 PR 关联：
参与讨论