Prhub

#32257 Revert "docker: install dynamo nightly in the dev image for rapid iteration/testing"

原始 PR 作者 Fridge003 合并时间 2026-07-24 05:51 文件变更 2 提交数 1 评论 1 代码增减 +4 / -38

执行摘要

回退 Docker 开发镜像中 Dynamo nightly 安装

PR body 直接引用回退原因:"Error: https://github.com/sgl-project/sglang/actions/runs/29969356548",即原始 #29634 的 CI 测试(Extra)失败,导致需要回退。

此 PR 为纯回退操作,逻辑简单,无需深入精读。建议关注未来重新引入该功能时需要解决的 CI 失败根因。

讨论亮点

该 PR 无 review 评论。唯一的一条 issue 评论来自 gemini-code-assist[bot],指出其对 GitHub 的代码审查已终止,与回退本身无关。

实现拆解

  1. Dockerfile 变更(docker/Dockerfile):删除约 30 行用于安装 ai-dynamo nightly 的脚本,包括:

    • 移除 ARG INSTALL_DYNAMO=0ARG DYNAMO_VERSION= 变量声明。
    • 移除根据 INSTALL_DYNAMO 标志从 PyPI 索引获取并安装最新 ai-dynamoRUN 命令。
    • 修改 gateway wheel 安装步骤:之前根据 INSTALL_DYNAMO 分支选择是否加 --no-deps,现在统一使用普通安装方式(去掉 if-else,直接执行 python3 -m pip install --force-reinstall /tmp/gateway_wheels/*.whl)。
    • 移除 framework_final 阶段末尾的 ARG INSTALL_DYNAMO=0 声明。
  2. CI 工作流变更(.github/workflows/release-docker-dev.yml):删除所有 --build-arg INSTALL_DYNAMO=1 的传递,即:

    • schedule 触发时:移除 --build-arg INSTALL_DYNAMO=1,只保留 USE_LATEST_SGLANG=1CMAKE_BUILD_PARALLEL_LEVEL
    • 手动/事件触发时:同理,移除 INSTALL_DYNAMO=1
  3. 无其他文件修改:回退仅涉及这两个基础设施文件,无源码或测试变更。

文件 模块 状态 重要度
docker/Dockerfile Docker modified 4.37
.github/workflows/release-docker-dev.yml CI modified 3.16

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

评论区精华

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

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

风险与影响

风险极低。此 PR 是回退操作,完全移除之前引入的变更。主要风险在于:

  • 如果未来需要重新引入 Dynamo nightly 集成,需要重新实现,可能丢失历史上下文。
  • 当前开发镜像中不再包含 ai-dynamo,依赖 Dynamo 测试的团队需要自行安装。
  • 用户/开发者:不再在 dev 镜像中包含 ai-dynamo nightly。依赖 Dynamo 进行集成测试的团队需要手动安装或依赖其他镜像。
  • CI/CD:release-docker-dev.yml 中不再传递 INSTALL_DYNAMO,构建时间减少(约 30+ 行脚本执行),但功能被移除。
  • 影响程度:较低,因为该功能是可选且未进入正式发布镜像。
CI 失败触发回退 纯基础设施变更 无测试覆盖

关联 Issue

#29634 docker: install dynamo nightly in the dev image for rapid iteration/testing

完整报告

参与讨论