执行摘要
更新 AMD ROCm Dockerfile 中 AITER 依赖的提交哈希,并移除已包含在上游版本的临时补丁 cherry-pick。变更仅涉及 docker/rocm.Dockerfile,无核心代码改动,风险低。
功能与动机
升级 AITER 依赖到新版本(提交 46e6c92b3eb33f64823aaa1ff39a14586b059ef5),该版本已包含之前需要手动 cherry-pick 的 aiter_mhc_pre 修复(b639cb63)。PR 清理了注释中明确标注“may be removed in next aiter upgrade”的临时补丁,保持构建流程干净。
实现拆解
- 更新 AITER 版本哈希:在 Dockerfile 的四个构建阶段(分别对应不同 ROCm 基础镜像)中,将
ENV AITER_COMMIT_DEFAULT 的值从旧哈希替换为新哈希。
- 移除过时 cherry-pick:删除
git cherry-pick --no-commit b639cb63bcac4672dce33a731fad042a65cb3649 命令以及其上方的注释行,因为新版本已原生包含该修复。
无关键源码片段。
评论区精华
"The upgrade of AITER_COMMIT_DEFAULT to 46e6c92b3eb33f64823aaa1ff39a14586b059ef5 makes the cherry-pick of b639cb63bcac4672dce33a731fad042a65cb3649 … obsolete … keeping the git cherry-pick command … will cause the Docker build to fail with an empty commit error." — gemini-code-assist[bot]
"@bingxche please clean up cherry-pick" — HaiShaw
"cleaned" — bingxche
风险与影响
- 风险:新版本 AITER 可能与现有环境不兼容,但 CI base test 通过,extra test 失败与本次变更关联不大。
- 影响:仅影响 AMD ROCm Docker 镜像构建。用户需重新构建镜像以使用新版本 AITER。
关联脉络
与之前的 PR #26544(修复 AITER git checkout 失败)同属 AMD 平台 Docker 构建稳定性改进系列,表明团队在持续优化 AMD 基础设施。
参与讨论