执行摘要
- 一句话:将 AMD Miles 的 nightly 构建与测试从 ROCm 7.2/7.0 迁移到 ROCm 10。
- 推荐动作:此 PR 是一个直接的 CI 基础设施迁移,值得快速审阅以确认配置细节。核心决策(迁移目标、镜像标签)已在上游 Issue #2854 中讨论并验证。对于关注 AMD 平台支持的工程师,这是一个重要的维护步骤。
功能与动机
上游 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)。
实现拆解
- 重命名与迁移 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,以反映新的构建目标。
- 迁移 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。
- CI 配套:此变更不涉及源代码、测试或模型逻辑的修改。它纯粹是 CI/CD 基础设施的调整,旨在将 nightly 构建和测试轨道与新的 ROCm 10 平台对齐。
关键文件:
.github/workflows/nightly-test-amd-miles-rocm10.yml(模块 测试调度;类别 infra;类型 rename-or-move): 核心变更文件,将 scheduled 测试工作流从 ROCm 7.2 迁移到 ROCm 10,包含镜像标签解析和测试调度逻辑。
.github/workflows/release-docker-amd-miles-rocm10-nightly.yml(模块 镜像构建;类别 infra;类型 rename-or-move): 将 nightly 构建工作流从已退役的 ROCm 7.0 轨道迁移到新的 ROCm 10 轨道,定义构建矩阵。
关键符号:未识别
评论区精华
唯一的审核评论来自 bingxche,提供了验证更新信息。该评论确认了 ROCm 10 Miles 镜像的构建/推送成功,并链接了使用该特定标签的 nightly 运行结果。这表明 PR 已在实际 CI 环境中得到验证,没有发现技术争议或未解决的疑虑。
风险与影响
- 风险:风险较低。变更仅限于 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 配置变更, 依赖上游仓库状态
关联脉络
- PR #38659 [AMD][CI] Make ROCm 10 the Default for AMD PR and Nightly Tests: 同属 AMD CI 基础设施演进脉络,该 PR 将 AMD PR 和 Nightly 测试的默认 ROCm 版本升级至 10,与本 PR 的 Miles nightly 迁移方向一致。
参与讨论