Prhub

#2734 docker: point MEGATRON_BRANCH back at miles-main

原始 PR 作者 yueming-yuan 合并时间 2026-08-25 04:01 文件变更 1 提交数 1 评论 0 代码增减 +1 / -1

执行摘要

Dockerfile 的 MEGATRON_BRANCH 回退到 miles-main

PR body 说明:临时分支名 miles-main-20260819 在 Megatron 侧已删除,而 miles-main 已指向升级后的树,因此需要将 Dockerfile 中的分支引用改回 miles-main,以修复构建失败。

此 PR 为简单的基础设施修复,不值得精读。核心决策是确保分支引用与上游仓库一致,建议关注后续是否有类似的临时分支引用清理。

讨论亮点

无实质性讨论,仅有 Claude bot 的自动提示和 Zhichenzzz 的批准。

实现拆解

  1. 修改 docker/Dockerfile 中的 ARG MEGATRON_BRANCH 值,从 miles-main-20260819 改为 miles-main。
  2. 无其他变更,因为 CI 默认分支、Dockerfile.rocm、docker-build.yml 的指纹和发布锁均未移动,它们会自动正确。
文件 模块 状态 重要度
docker/Dockerfile 构建配置 modified 2.78

关键源码片段

docker/Dockerfile infrastructure

修改了 MEGATRON_BRANCH 参数,是构建配置的关键部分

# docker/Dockerfile
ARG SGLANG_BRANCH=sglang-miles
ARG SGLANG_COMMIT=""
ARG MEGATRON_REPO=radixark/Megatron-LM
# 关键变更:分支名从 'miles-main-20260819' 回退到 'miles-main'
# 因为临时分支已删除,miles-main 已指向升级后的树
ARG MEGATRON_BRANCH=miles-main
# Empty means the branch HEAD at build time; release builds set it to freeze one commit.
ARG MEGATRON_COMMIT=""

评论区精华

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

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

风险与影响

风险极低,仅修改 Dockerfile 中的一个 ARG 默认值。但需确认 miles-main 分支确实存在且与预期版本一致,否则可能导致构建失败或使用错误的 Megatron 版本。

影响范围仅限 Docker 构建过程,使基于 Dockerfile 的构建使用 miles-main 分支。对最终用户无直接影响,但确保 CI 和发布构建使用正确的 Megatron-LM 代码。

构建配置变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论