Prhub

#6215 [ci] chore: bump trtllm to 1.3.0rc13 and verl to v0.7.1

原始 PR 作者 Superjomn 合并时间 2026-04-30 09:58 文件变更 2 提交数 1 评论 2 代码增减 +4 / -4

执行摘要

bump trtllm 1.3.0rc13 和 verl v0.7.1

需要将 TRT-LLM 基础镜像升级到 1.3.0rc13 并同步 verl 版本至 v0.7.1,以确保 CI 使用最新的依赖和镜像。

PR 内容简洁明确,值得快速浏览以了解版本升级流程。但审核中提出的 Dockerfile 优化建议(移除冗余安装、合并层)值得在后续类似 PR 中采纳。

讨论亮点

审核机器人 gemini-code-assist[bot] 提出两条建议:一是 Dockerfile 中第 45 行多余的 pip install verl 步骤(随后被卸载)可以删除以优化构建效率;二是建议使用 --no-cache-dir 并合并 RUN 命令以减少镜像层数。同时指出同仓库的 docker/Dockerfile.stable.sglang 仍引用 v0.7.0,建议一并更新。这些建议未被采纳或回复,但 PR 已由 wuxibin89 批准合并,说明团队可能认为当前改动已满足需求或将在后续优化。

实现拆解

  1. 修改 docker/Dockerfile.stable.trtllm 中的 TRTLLM_BASE_IMAGE 参数为 nvcr.io/nvidia/tensorrt-llm/release:1.3.0rc13,并将 verlverl[mcore] 的 pip 安装来源从 v0.7.0 更新为 v0.7.1
  2. 修改 .github/workflows/e2e_ppo_grpo_trainer_trtllm.yml 中的 IMAGE 环境变量,将镜像标签从 trtllm1.3.0rc10 更新为 trtllm1.3.0rc13,与 Dockerfile 同步。
  3. 无其他源码或测试变更。
文件 模块 状态 重要度
docker/Dockerfile.stable.trtllm 部署脚本 modified 3.08
.github/workflows/e2e_ppo_grpo_trainer_trtllm.yml CI modified 2.95

关键源码片段

docker/Dockerfile.stable.trtllm infrastructure

Dockerfile 中更新了 TRT-LLM 基础镜像版本和 verl 依赖版本,是本次变更的核心文件。

# 基础镜像版本从 1.3.0rc10 升级至 1.3.0rc13
ARG TRTLLM_BASE_IMAGE=nvcr.io/nvidia/tensorrt-llm/release:1.3.0rc13
​
# ...(中间部分忽略)...# 安装 verl 依赖,版本从 v0.7.0 升级至 v0.7.1
RUN pip install git+https://github.com/verl-project/verl.git@v0.7.1
RUN pip uninstall -y verl
RUN pip install "verl[mcore] @ git+https://github.com/verl-project/verl.git@v0.7.1"
RUN pip uninstall -y verl

评论区精华

Dockerfile 冗余安装优化 设计

gemini-code-assist[bot] 指出第 45 行的 `pip install verl` 是冗余的,因为第 47 行会重新安装包含 mcore 扩展的版本,建议删除以简化构建。

结论:未在 PR 中处理,但 PR 已批准合并。 · unresolved

Dockerfile 构建层优化 设计

gemini-code-assist[bot] 建议使用 `--no-cache-dir` 并将安装与卸载合并为单个 RUN 命令以减少层数和镜像大小。

结论:未在 PR 中处理,但 PR 已批准合并。 · unresolved

其他 Dockerfile 版本一致性 other

审核机器人指出 `docker/Dockerfile.stable.sglang` 仍引用 v0.7.0,建议同步更新。

结论:未在本次 PR 处理,但提示了后续维护方向。 · unresolved

风险与影响

风险较低:仅更改镜像标签和依赖版本,不涉及业务逻辑。但需确认新镜像版本与 CI 环境的兼容性,以及是否遗漏其他 Dockerfile(如 sglang)的同步,可能造成版本不一致。

影响范围有限:仅影响 trtllm 相关 CI 构建流程和 Docker 镜像构建,其他模块无影响。对用户和系统无直接功能影响。

缺少测试覆盖 可能的版本不一致

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论