# PR #5748 完整报告

- 仓库：`verl-project/verl`
- 标题：[ci] chore: delete install current repository for npu ci
- 合并时间：2026-03-26 02:43
- 原文链接：http://prhub.com.cn/verl-project/verl/pull/5748

---

## 执行摘要
本 PR 通过移除 NPU CI 流水线中的显式依赖安装步骤并添加本地镜像配置，优化了 CI 执行速度和可靠性，但引入了对 Docker 镜像预装内容的依赖风险，影响多个测试工作流和贡献者文档。

## 功能与动机
变更动机是优化 NPU CI 的性能，假设依赖项已预装在 Docker 镜像中，以减少安装时间和网络延迟。review 评论指出：“This change removes the explicit installation of dependencies, assuming they are pre-installed in the Docker image. While this is a valid optimization for CI...”，这体现了对 CI 效率的优先考虑。

## 实现拆解
实现主要修改了 11 个文件：
- **CI 工作流文件 **（如 `.github/workflows/e2e_ascend.yml`）：添加了配置 APT 和 pip 镜像的步骤，例如：
  ```yaml
  - name: config mirrors
    run: |
      sed -Ei 's@(ports|archive).ubuntu.com@192.168.9.104:32685@g' /etc/apt/sources.list
      pip config set global.index-url http://192.168.9.104:32303/pypi/simple
  ```
 并将安装命令从 `pip install -r requirements-npu.txt - pip install -e .` 改为 `pip install --no-deps -e .`。

- **文档文件 **（`docs/ascend_tutorial/contribution_guide/ascend_ci_guide_zh.rst`）：更新安装步骤，移除依赖安装说明。
- **脚本文件 **（`examples/grpo_trainer/run_qwen3_235b_megatron_npu.sh`）：修复语法错误，确保配置一致性。

## 评论区精华
review 讨论的核心点是设计权衡：gemini-code-assist[bot] 建议添加注释以解释依赖预装假设，避免贡献者复制工作流时产生混淆。原评论强调：“...it makes the workflow example less explicit and potentially confusing for contributors. Someone copying this template might not realize there's a hidden dependency...”。最终 PR 被批准，但文档未采纳建议注释，这可能留下清晰度隐患。

## 风险与影响
- **技术风险**：如果 Docker 镜像未预装正确依赖（如 requirements-npu.txt 中的包），CI 将失败；硬编码镜像 IP（192.168.9.104）可能失效；文档不明确可能导致贡献者误配置。
- **影响范围**：直接影响 NPU CI 流水线的执行效率，可能减少安装时间，但增加了对镜像维护的依赖；贡献者需注意隐含假设，团队需确保镜像更新。

## 关联脉络
与此 PR 相关的历史 PR 包括：
- #5680：新增 NPU 支持，扩展了 CI 测试范围。
- #5724：升级 CI 依赖，调整配置以解决兼容性问题。
这些 PR 共同反映了仓库在 NPU CI 优化和依赖管理方面的持续演进，本 PR 是这一趋势中的性能调优步骤。