Prhub

#21089 Fix jit-kernel unit test CI dependency install in pr-test workflow

原始 PR 作者 merrymercy 合并时间 2026-03-22 13:13 文件变更 1 提交数 3 评论 3 代码增减 +2 / -4

bugfix

执行摘要

修复 pr-test 工作流中 JIT-kernel 单元测试的依赖安装配置错误。

根据PR body描述，变更动机是修复CI依赖安装步骤，使它们匹配预期配置，以避免依赖错误导致测试失败。引用原文：'Updates .github/workflows/pr-test.yml so CUDA dependency install steps match the intended profiles.'

该PR变更简单直接，不建议深入阅读，除非您负责CI维护或遇到类似依赖安装问题。可快速浏览patch_excerpt确认配置调整，无需精读代码逻辑。

讨论亮点

本PR无review评论，因此没有技术讨论或争议点。

实现拆解

实现方案仅修改一个文件：.github/workflows/pr-test.yml。关键改动包括：

  1. 在B200 diffusion job中，从安装命令移除IS_BLACKWELL=1环境变量，依赖CUSTOM_BUILD_SGL_KERNEL参数和脚本默认行为。
  2. 在JIT-kernel unit test job中，将安装命令从bash scripts/ci/cuda/ci_install_dependency.sh改为bash scripts/ci/cuda/ci_install_dependency.sh diffusion，指定使用diffusion配置。
  3. 在5090 job中，移除env: IS_BLACKWELL: "1"的冗余环境变量块。
文件 模块 状态 重要度
.github/workflows/pr-test.yml CI/CD modified 4.0

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

评论区精华

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

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

风险与影响

风险较低，主要风险包括：

  • CI配置变更可能导致作业失败，特别是如果ci_install_dependency.sh脚本的行为未正确验证。
  • 删除IS_BLACKWELL环境变量可能影响B200扩散作业的兼容性，但PR body解释为冗余，依赖其他机制。
  • 需要确保测试计划通过验证以避免后续CI中断。

影响范围仅限于CI/CD流程，具体为pr-test工作流的执行。对最终用户、核心系统或团队日常开发无直接影响。影响程度为低，主要改善CI的稳定性和准确性。

CI 配置变更 依赖安装调整

关联 Issue

未识别关联 Issue

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

完整报告

Markdown PDF

参与讨论