Prhub

#37495 [AMD] ci: move the miles nightlies from rocm700 to rocm10

原始 PR 作者 XinyuJiangCMU 合并时间 2026-09-10 10:37 文件变更 2 提交数 3 评论 1 代码增减 +6 / -7

执行摘要

将 AMD Miles 的 nightly 构建与测试从 ROCm 7.2/7.0 迁移到 ROCm 10。

上游 SGLang ROCm 7.0 基础镜像已停止构建,而基于 ROCm 10 的 MI35X 基础镜像现已每日构建,并支持 Python 3.12 和基于 wheel 的 ROCm SDK。此外,ROCm 10 不再出现 ROCm 7.2 上观察到的 VMM-pause 或 IPC-export 内存问题,因此旧的 ROCr 和 legacy-IPC 解决方案不再需要。此 PR 的动机是淘汰已退役的 rocm700 变体,并启用新的 rocm10-mi35x 变体,以跟随上游变更(关联 Issue #2854)。

此 PR 是一个直接的 CI 基础设施迁移,值得快速审阅以确认配置细节。核心决策(迁移目标、镜像标签)已在上游 Issue #2854 中讨论并验证。对于关注 AMD 平台支持的工程师,这是一个重要的维护步骤。

讨论亮点

唯一的审核评论来自 bingxche,提供了验证更新信息。该评论确认了 ROCm 10 Miles 镜像的构建/推送成功,并链接了使用该特定标签的 nightly 运行结果。这表明 PR 已在实际 CI 环境中得到验证,没有发现技术争议或未解决的疑虑。

实现拆解

  1. 重命名与迁移 nightly 构建工作流:将 .github/workflows/release-docker-amd-miles-rocm700-nightly.yml 重命名为 .github/workflows/release-docker-amd-miles-rocm10-nightly.yml。在工作流内部,将名称和矩阵变体从 rocm700-mi35x 和 rocm700-mi30x 更改为 rocm10-mi35x,以反映新的构建目标。
  2. 迁移 scheduled 测试工作流:将 .github/workflows/nightly-test-amd-miles-rocm720.yml 重命名为 .github/workflows/nightly-test-amd-miles-rocm10.yml。在工作流内部,将名称、输入参数描述和镜像解析逻辑中的 rocm720 引用更新为 rocm10,例如将基础镜像标签从 miles-rocm720-mi35x 改为 miles-rocm10-mi35x。
  3. CI 配套:此变更不涉及源代码、测试或模型逻辑的修改。它纯粹是 CI/CD 基础设施的调整,旨在将 nightly 构建和测试轨道与新的 ROCm 10 平台对齐。
文件 模块 状态 重要度
.github/workflows/nightly-test-amd-miles-rocm10.yml 测试调度 renamed 4.14
.github/workflows/release-docker-amd-miles-rocm10-nightly.yml 镜像构建 renamed 4.0

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

评论区精华

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

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

风险与影响

风险较低。变更仅限于 CI 工作流配置,不涉及运行时代码、内核或核心调度逻辑。主要风险在于 CI 配置错误可能导致 nightly 构建或测试失败,但此风险已通过 bingxche 的验证评论得到缓解。确保 radixark/miles@main 上存在 rocm10-mi35x 变体是成功迁移的前提,该条件已满足。

影响范围限于 AMD 平台的 CI/CD 基础设施。对于开发团队,这意味着 nightly 构建和测试将使用新的 ROCm 10 环境,可能影响对 AMD 硬件的兼容性验证。对于系统,CI 流水线将继续运行,但目标平台从 ROCm 7.2/7.0 升级到 ROCm 10。此变更有助于保持与上游硬件/软件演进的一致性。

CI 配置变更 依赖上游仓库状态

关联 Issue

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

完整报告

参与讨论