Prhub

#23607 [AMD] upd local registry address

原始 PR 作者 yctseng0211 合并时间 2026-04-24 12:07 文件变更 4 提交数 1 评论 1 代码增减 +5 / -5

执行摘要

更新 AMD CI 本地 Docker 镜像仓库地址

本地Docker镜像仓库的地址发生变更,需要同步更新CI配置以维持AMD测试基础设施的正常运行。PR body未提供更多细节,但从变更内容看是基础设施运维层面的地址迁移。

该PR属于运维层面的常规更新,技术价值低,无需深入阅读。

讨论亮点

该PR没有review评论。只有一个来自AI助手的每日配额限制提醒,无实际讨论。

实现拆解

  1. 更新 scripts/ci/amd/amd_ci_start_container.sh:将 LOCAL_DOCKER_REGISTRY 变量值从 172.29.8.23:5000 改为 10.245.143.50:5000。该变量在脚本中用于拉取和推送镜像。
  2. 更新 scripts/ci/amd/amd_ci_start_container_disagg.sh:同上,修改 LOCAL_DOCKER_REGISTRY 变量值。
  3. 更新 .github/workflows/release-docker-amd-nightly.yml:在 "Mirror rocm/sgl-dev to local registry" 步骤中,将目标地址 dst 从旧地址改为新地址。
  4. 更新 .github/workflows/release-docker-amd-rocm720-nightly.yml:同上,修改镜像目标地址。
文件 模块 状态 重要度
scripts/ci/amd/amd_ci_start_container.sh CI 脚本 modified 3.13
.github/workflows/release-docker-amd-nightly.yml 工作流 modified 2.55
.github/workflows/release-docker-amd-rocm720-nightly.yml 工作流 modified 2.55
scripts/ci/amd/amd_ci_start_container_disagg.sh CI 脚本 modified 2.55

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

评论区精华

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

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

风险与影响

风险极低。变更仅为地址字符串替换,不涉及逻辑改动。若新地址不可达,可能导致镜像拉取/推送失败,但CI会在早期阶段暴露此问题。

仅影响AMD CI基础设施的本地镜像仓库通信。对用户和其他平台无影响。若新仓库地址配置正确,可正常恢复CI流程;否则需要回滚或修复地址。

基础设施依赖更新 地址变更可能导致 CI 失败

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论