Prhub

#6618 [ci] fix: remove uv usage from A2 vlm_rl_job, use pip instead

原始 PR 作者 KadenZhang3321 合并时间 2026-06-06 10:21 文件变更 8 提交数 1 评论 2 代码增减 +43 / -12

执行摘要

移除 A2 job 中的 uv 改用 pip

PR 描述指出:The vlm_rl_job runs on linux-aarch64-a2b3-8 (A2 Ascend 910B machines) which do not support uv。其他 12 个 A2 job 已正确使用 pip,此 PR 是唯一遗留的误用 uv 的 job。

该 PR 为纯 CI 修复,逻辑简单,可快速合入。值得关注的点:团队在维护多硬件 CI 时通过环境变量统一缓存策略的做法,可供其他基础设施 PR 参考。

讨论亮点

无有效讨论。Reviewer wucong25 直接批准,无评论。

实现拆解

  1. .github/workflows/e2e_ascend.ymlvlm_rl_job 中移除 Install uv 步骤和所有 UV_* 环境变量,将 uv pip listuv pip install 替换为 pip 等效命令,并添加 PIP_INDEX_URLPIP_TRUSTED_HOST 以使用内部 PyPI 缓存。
  2. 在其他 7 个 workflow 文件中(如 nightly_ascend.ymlreward_model_sglang_ascend.yml 等)统一添加 UV_INDEX_URLPIP_INDEX_URLPIP_TRUSTED_HOST 环境变量,确保保留 uv 的 job 也能使用内部缓存;同时修复 e2e_ppo_trainer_megatron_sglang_ascend.yml 中一个遗漏的 uv pip install 命令为 pip install
  3. 所有变更仅涉及 CI 部署配置,未改动任何核心业务逻辑或测试代码。总变更量 +43/-12 行,覆盖 8 个文件。
文件 模块 状态 重要度
.github/workflows/e2e_ascend.yml CI 配置 modified 4.23
.github/workflows/nightly_ascend.yml CI 配置 modified 3.57
.github/workflows/e2e_ppo_trainer_megatron_sglang_ascend.yml CI 配置 modified 3.51

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

评论区精华

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

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

风险与影响

风险极低。pip 是 Python 标准包管理器,已在其他 A2 job 中验证可用;内部缓存配置(PIP_INDEX_URL、PIP_TRUSTED_HOST)与其他 job 保持一致,不会引入新问题。若后续 A2 机器环境有变,可能需再次调整包管理器选择。

影响范围仅限于 Ascend A2 集群的 CI 流程(vlm_rl_job),使其能正常安装依赖,不涉及用户侧功能、性能或 API 变更。对其他硬件平台(如 GPU、NPU 非 A2 型号)无影响。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论