Prhub

#29394 [AMD] [Docker] Update MoRI to v1.2.1

原始 PR 作者 Duyi-Wang 合并时间 2026-07-06 10:37 文件变更 1 提交数 1 评论 2 代码增减 +1 / -1

执行摘要

更新 ROCm Docker 镜像的 MoRI 依赖版本

关联 Issue #27669 已将 MoRI 更新至 v1.2.0,而 v1.2.1 标签包含了针对 ROCm 7.1.4 的支持修复,因此需要将 Docker 构建中的 MoRI 版本同步提升,使 ROCm 用户能够受益于该修复。

值得关注的是维护团队对依赖管理规范的坚持:Dockerfile 中统一使用提交 SHA 并附加版本注释的做法值得借鉴。

讨论亮点

机器人审查者 gemini-code-assist[bot] 建议使用不可变的提交 SHA 而非版本标签,以确保 Docker 构建的可重复性和安全性,并指出该 Dockerfile 中其他依赖(如 TRITON_COMMITMOONCAKE_COMMIT)均使用 SHA 而非标签。项目维护者 HaiShaw 同意了该意见,并要求在 v1.2.1 旁附加提交 SHA 注释。最终合并版本采纳了此建议。

实现拆解

  1. 修改 docker/rocm.Dockerfile 第 110 行:将 MORI_COMMIT 的值从 v1.2.0 的提交 SHA 改为 v1.2.1 版本标签。
  2. 该变更在 Review 讨论后,最终合并时实际保留了提交 SHA(e31d426a13e96e1cbff96a1c904d291aefe8c46a)并附带注释说明,兼顾了可追溯性与安全性。
文件 模块 状态 重要度
docker/rocm.Dockerfile Docker modified 2.78

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

评论区精华

使用提交 SHA 而非版本标签以确保构建可重复性 正确性

机器审查者建议将 `MORI_COMMIT` 改为不可变的提交 SHA,以符合 Docker 构建最佳实践,并引用该 Dockerfile 中其他依赖的写法。

结论:采纳:最终版本保留了提交 SHA,并在注释中标明对应版本 v1.2.1。 · 已解决

风险与影响

风险极低:仅修改 Dockerfile 中的一行依赖版本引用。若 v1.2.1 存在兼容性问题,可能影响基于 ROCm 的 MoRI 功能(如 EP 和分解模式),但该版本已在关联 Issue #27669 中通过 CI 测试。

直接影响使用 ROCm Docker 镜像构建的 SGLang 用户:升级后 MoRI 将打上 v1.2.1 修复,对 ROCm 7.1.4 的支持更稳定。无代码层面影响,无需用户手动操作。

依赖版本升级 仅影响 ROCm Docker 构建

关联 Issue

#27669 [AMD] Update MoRI to v1.2.0

完整报告

参与讨论