Prhub

#5523 [ci] chore: update npu ci to vllm013

原始 PR 作者 yyyy2000 合并时间 2026-03-12 10:00 文件变更 16 提交数 13 评论 6 代码增减 +139 / -120

执行摘要

升级 NPU CI 至 vLLM 0.13.0,更新 CANN 版本为 8.5.0 并优化测试配置。

根据PR标题和review讨论,动机是升级NPU CI以支持vLLM 0.13.0。review评论中提到'vllm0.13.0 ok',表明需要更新CI以适配新版本vLLM,同时将CANN从8.5.RC1升级到8.5.0以保持环境一致性和稳定性。

建议技术管理者快速浏览此PR以了解CI升级细节,工程师可关注Dockerfile重命名和测试配置变化,确保本地环境同步。PR包含有价值的CI维护经验,但无需深入代码逻辑。

讨论亮点

review中主要讨论点:

1) gemini-code-assist[bot]指出文档链接更新但Dockerfile未重命名,可能导致链接失效;作者在提交中通过重命名Dockerfile解决了此问题。
2) wucong25询问删除'export PYTHONPATH=$PYTHONPATH:/Megatron-LM'行和'disable_mm_preprocessor_cache=True'参数的原因,表明对变更细节的关注,但无明确结论,可能已通过后续提交优化。讨论聚焦于正确性和设计权衡。

实现拆解

实现方案分为三个部分:

1) CI工作流更新:修改了8个GitHub Actions YAML文件(如e2e_ascend.yml、e2e_ppo_trainer_megatron_vllm_2_ascend.yml),将容器镜像标签从'verl-8.3.rc1-910b'更新为'verl-8.5.0-910b',并调整测试步骤,如在npu_unit_tests.yml中添加激活卸载测试。
2) Dockerfile重命名:将'docker/ascend/Dockerfile.ascend_8.5.rc1_a2/a3'重命名为'docker/ascend/Dockerfile.ascend_8.5.0_a2/a3'以反映CANN版本。
3) 文档和脚本更新:修订了ascend教程文档(如ascend_ci_guide_zh.rst)和测试脚本(如run_grpo_qwen25-vl-3b-instruct_fsdp_npu.sh),确保版本和配置一致。

文件 模块 状态 重要度
.github/workflows/e2e_ppo_trainer_megatron_vllm_2_ascend.yml CI modified 6.0
docker/ascend/Dockerfile.ascend_8.5.0_a2 Docker renamed 5.0
docs/ascend_tutorial/dockerfile_build_guidance.rst Documentation modified 4.0

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

评论区精华

Dockerfile 重命名与文档一致性 正确性

gemini-code-assist[bot] 指出文档链接已更新但 Dockerfile 未重命名,将导致链接失效。

结论:作者在提交中重命名了 Dockerfile 以匹配文档,问题已解决。 · 已解决

删除 CI 脚本中的配置行 question

wucong25 询问为什么删除 export PYTHONPATH 行和 disable_mm_preprocessor_cache 参数。

结论:无明确结论,可能已通过优化移除,但未在讨论中详细说明。 · unresolved

风险与影响

技术风险包括:

1) CI配置错误:镜像标签或路径错误(如.github/workflows/e2e_ppo_trainer_megatron_vllm_2_ascend.yml中大量改动)可能导致CI作业失败,影响开发流程。
2) 兼容性问题:CANN 8.5.0可能引入新行为,需确保测试覆盖;
3) 文档不一致:如果Dockerfile重命名未同步,用户可能访问错误链接。

影响范围:

1) 对用户:无直接API变更,但CI稳定性影响团队开发效率和NPU环境测试;
2) 对系统:NPU CI环境升级到新版本,提升测试可靠性和支持vLLM 0.13.0特性;
3) 对团队:确保新版本兼容性,减少未来集成问题。影响程度中等,主要限于CI基础设施。

CI 配置变更 文档不一致 兼容性风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论