执行摘要
更新 AMD 本地 registry 地址并移除旧镜像作业
AMD CI 和发布流程需要切换到新的本地 registry 地址,同时清理已废弃的 Miles 镜像作业,减少维护负担。
PR 变更简单直接,适合快速合并。建议合并后监控一次 AMD CI 运行确认新 registry 正常工作。
无 review 评论,直接由 bingxche 批准合并,变更清晰无争议。
AMD CI 和发布流程需要切换到新的本地 registry 地址,同时清理已废弃的 Miles 镜像作业,减少维护负担。
PR 变更简单直接,适合快速合并。建议合并后监控一次 AMD CI 运行确认新 registry 正常工作。
无 review 评论,直接由 bingxche 批准合并,变更清晰无争议。
scripts/ci/amd/amd_ci_start_container.sh、scripts/ci/amd/amd_ci_start_container_disagg.sh、release-docker-amd-nightly.yml 和 release-docker-amd-rocm720-nightly.yml 中将所有 10.245.143.50:5000 替换为 10.44.14.109:5000。release-docker-amd-miles-rocm700-nightly.yml 和 release-docker-amd-miles-rocm720-nightly.yml 中的 push_local_registry 作业和相关步骤(如镜像标签上传、下载、推送到本地 registry),每个文件减少约 50 行。rg -n "10\.245\.143\.50:5000" . 确认无残留旧地址,并检查 YAML 语法和 shell 脚本无错误。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
.github/workflows/release-docker-amd-miles-rocm700-nightly.yml |
CI 配置 | modified | 4.9 |
.github/workflows/release-docker-amd-miles-rocm720-nightly.yml |
CI 配置 | modified | 4.9 |
scripts/ci/amd/amd_ci_start_container.sh |
CI 脚本 | modified | 3.13 |
scripts/ci/amd/amd_ci_start_container_disagg.sh |
CI 脚本 | modified | 2.95 |
.github/workflows/release-docker-amd-nightly.yml |
CI 配置 | modified | 2.55 |
.github/workflows/release-docker-amd-rocm720-nightly.yml |
CI 配置 | modified | 2.55 |
分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。
当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。
风险极低:仅涉及 CI/CD 基础设施配置变更(registry 地址)和删除不再使用的作业。若新 registry 不可达,可能导致 AMD 镜像拉取失败,但已有验证流程确保地址有效。删除 Miles 作业不会影响现有发布流程。
影响范围限于 AMD 相关的 CI 和发布流水线。开发者无感知,用户无影响。变更后 registry 流量指向新地址,旧 registry 将不再接收推送。
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论