执行摘要
此 PR 回退了先前在 Docker 开发镜像中集成 ai-dynamo nightly 的变更(#29634),原因是 CI 运行失败。变更仅涉及 docker/Dockerfile 和 .github/workflows/release-docker-dev.yml,移除约 34 行代码,风险极低。
功能与动机
PR body 直接引用失败 CI 运行链接(Run #29969356548),表明原始功能引入后 CI 测试(Extra)未通过,因此选择回退以恢复稳定状态。动机是功能稳定性而非设计决策。
实现拆解
-
Dockerfile 清理 (docker/Dockerfile)
- 删除所有与 INSTALL_DYNAMO 相关的 ARG 声明和条件执行逻辑。之前通过动态解析 PyPI 索引获取最新 nightly 版本并安装,回退后不再执行。
- 统一 gateway wheel 安装命令,移除之前分叉(if [ "$INSTALL_DYNAMO" = "1" ])的逻辑,直接使用 python3 -m pip install --force-reinstall /tmp/gateway_wheels/*.whl。
-
CI 工作流简化 (.github/workflows/release-docker-dev.yml)
- 删除所有构建参数中的 --build-arg INSTALL_DYNAMO=1,无论触发事件是 schedule 还是手动/事件触发。
-
无其他配套变更:回退不涉及测试、配置或文档更新。
无需要突出展示的关键源码片段,因为变更主要是删除代码和简化条件。
评论区精华
无 review 评论。仅有一条机器人评论关于 Gemini Code Assist 终止,与回退无关。
风险与影响
- 回归风险:几乎为零,因为移除的是可选功能。
- 影响范围:仅影响 dev 镜像的构建。之前依赖 dev 镜像中自动安装
ai-dynamo 的团队可能需要手动安装。
- CI 稳定性:回退后可消除由 Dynamo 集成引入的 CI 失败,提升 CI 可靠性。
关联脉络
- #29634(原始 PR):引入了 Dynamo nightly 安装,本 PR 为其逆操作。
- 与当前仓库中其他近期 PR(如 #32014 创建 Rust workspace、#32160 内核搬迁)无直接关联,属于独立的 Docker/CI 基础设施调整。
该回退为临时措施,未来可能以更稳定方式重新引入 Dynamo 集成。
参与讨论