Prhub

#30587 [AMD-miles] Bump miles rocm700-mi35x base image and migrate nightly test onto it

原始 PR 作者 JessicaJiang-123 合并时间 2026-07-10 23:47 文件变更 2 提交数 1 评论 1 代码增减 +7 / -5

执行摘要

AMD MI35x 夜间测试镜像升级与工作流迁移

维护 AMD 集群的持续集成基础设施,确保夜间测试使用最新的维护镜像。PR body 指出 rocm700-mi35x 是活跃维护的镜像,需要升级基础版本并迁移测试工作流。

值得阅读以了解 AMD 基础设施的维护实践,尤其是 Docker 构建参数化的工作流设计。

讨论亮点

无讨论:PR 无 review 评论,仅有一个 reviewer 批准。

实现拆解

  1. 工作流重命名:将 nightly-test-amd-miles-rocm720.yml 重命名为 nightly-test-amd-miles-rocm700.yml,并更新所有相关引用从 rocm720rocm700
  2. 基础镜像升级:在 release-docker-amd-miles-rocm700-nightly.yml 中将 sglang_image_tagv0.5.10-rocm700-mi35x 更新为 v0.5.14-rocm700-mi35x-20260627,并新增 sglang_image_repo 变量 rocm/sgl-dev
  3. Docker 构建灵活性:在构建步骤中通过 --build-arg SGLANG_IMAGE_REPO 传递镜像仓库参数,允许在需要时覆盖默认仓库 lmsysorg/sglang
文件 模块 状态 重要度
.github/workflows/nightly-test-amd-miles-rocm700.yml CI 工作流 renamed 4.23
.github/workflows/release-docker-amd-miles-rocm700-nightly.yml CI 工作流 modified 3.21

关键源码片段

.github/workflows/nightly-test-amd-miles-rocm700.yml rename-or-move

核心工作流文件,从 ROCm 7.2 迁移到 ROCm 7.0,并更新镜像标签解析逻辑。

# nightly-test-amd-miles-rocm700.yml
name: Nightly Test Miles (AMD ROCm 7.0)
on:
  schedule:
    # 定时触发
  workflow_dispatch:
    inputs:
      image_tag:
        description: "Override miles image tag (e.g. miles-rocm700-mi35x-20260621); empty = newest by date"
        required: false
        type: string
        default: ""
jobs:
  test:
    # ...
    - name: Resolve newest miles ROCm 7.0 image
      run: |
        repo="rocm/sgl-dev"
        base="miles-rocm700-mi35x"  # 之前是 miles-rocm720-mi35x
        tag="${{ inputs.image_tag }}"
        # 如果未指定 tag,则从今天往回查找最新的 dated tag
        if [ -z "${tag}" ]; then
          # Walk back from today; the build job pushes a dated tag at 12:00 UTC.

评论区精华

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

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

风险与影响

风险较低。变更仅涉及 CI 配置文件,没有代码逻辑变化。主要风险是新的基础镜像可能引入兼容性问题,但被限制在夜间测试流程中,不影响生产发布。

  • 影响范围:仅影响 AMD MI35x 的夜间测试工作流和 Docker 镜像发布。
  • 影响程度:低,用户无感知,团队需要确认新镜像上的测试通过。
基础设施变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论