Prhub

#38694 [AMD][CI] Drop the dead miles ROCm 7.0 nightly image build

原始 PR 作者 michaelzhang-ai 合并时间 2026-09-10 15:00 文件变更 1 提交数 1 评论 0 代码增减 +0 / -61

执行摘要

删除已废弃的 ROCm 7.0 Miles 夜间 Docker 镜像构建工作流,修复 CI 失败。

上游仓库 radixark/miles#2854 合并后删除了 rocm700-mi35x 和 rocm700-mi30x 变体。当前工作流仍尝试构建这些已退役的变体,导致 docker/build.py --variant rocm700-mi35x 命令失败。该失败每日消耗 CI 资源并在 MILES dashboard 上产生红色构建记录,属于无效噪音。PR body 引用了具体的失败 run 和错误信息。

此 PR 是一个标准、低风险的 CI 基础设施清理操作。由于其目的明确(修复已知的每日 CI 失败)、验证充分且无副作用,建议快速审查并合并。对于维护 CI 清洁度和节省计算资源具有明确价值。

讨论亮点

PR 无任何讨论评论,仅获得维护者 HaiShaw 的直接批准。这表明变更是一个明确的、无争议的清理操作,动机清晰(修复已知的 CI 失败),且验证充分(确认无引用、无影响)。

实现拆解

  1. 变更入口与核心动作:删除文件 .github/workflows/release-docker-amd-miles-rocm700-nightly.yml。该文件定义了名为 Release Docker Images Nightly Miles ROCm7.0 (AMD) 的工作流,负责每日构建并推送 rocm700-mi35x 和 rocm700-mi30x 两个 Docker 镜像标签。
  2. 配置与触发器清理:删除了该工作流的所有触发器(workflow_dispatch 和 schedule cron)、并发控制、权限设置、作业定义(publish)及所有步骤。这清除了所有对已退役变体的引用。
  3. 影响范围确认:通过 rg 'miles-rocm700' 验证,仓库内无其他工作流、脚本或文档消费 rocm/sgl-dev:miles-rocm700-* 标签。同时,release-docker-amd-miles-rocm720-nightly.yml 工作流不受影响,因为它构建的是 rocm720-mi35x 变体。删除操作不影响源码、测试或运行时逻辑。
文件 模块 状态 重要度
.github/workflows/release-docker-amd-miles-rocm700-nightly.yml CI 工作流 removed 5.21

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

评论区精华

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

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

风险与影响

  1. 回归风险极低:变更仅删除一个已失效的 CI 工作流文件。由于该工作流当前已无法成功运行,删除它不会改变任何现有功能或测试结果。
  2. 无功能或性能影响:不涉及任何 Python 源码、内核代码或运行时配置。仅清理 CI 基础设施。
  3. 依赖关系:与 PR #37495(为 ROCm 10 添加替代工作流)逻辑关联,但两者可独立合并。删除旧工作流不阻塞新工作流的部署。

对用户:无直接影响。用户不会感知到此变更。
对系统:积极影响。消除每日两次的 CI 失败(amd-docker-scale job),减少 CI 资源浪费和 dashboard 噪音,提升 CI 系统的整体健康度。
对团队:简化 CI 配置维护,移除一个已过时且无用的组件。

CI 配置变更

关联 Issue

#2854 [AMD] docker: retire the rocm700 variants, add a rocm10-mi35x variant

完整报告

参与讨论