Prhub

#47338 [Docker] Remove unused Dockerfile.nightly_torch

原始 PR 作者 atalman 合并时间 2026-07-02 07:19 文件变更 1 提交数 1 评论 0 代码增减 +0 / -326

执行摘要

删除无人引用的 Dockerfile.nightly_torch

PR body 指出,Dockerfile.nightly_torch 在 vLLM 仓库内的 .buildkite/ pipeline、docker/*.hcl bake 文件、image_build*.sh 脚本、.github/workflows 及文档中均无引用,同时在 vllm-project/ci-infra 仓库中也无任何引用。构建 torch nightly 镜像现在通过主 docker/Dockerfile 配合 PYTORCH_NIGHTLY=1 实现,因此该独立文件已完全被取代。

该 PR 是一次低风险清理,变更简单明确,无需深度审查。开发者可借此了解 PYTORCH_NIGHTLY 构建参数的存在,但不必花费大量时间阅读。

讨论亮点

无实质性 Review 讨论。机器人 claude[bot] 因 PR 来自 fork 自动禁用手动审查;khluu 批准了变更。

实现拆解

实现过程分为两步:

  1. 确认文件已死:通过全局搜索 grep -r Dockerfile.nightly_torch 确认仓库内无任何引用,并检查了 ci-infra 仓库确认无外部依赖。
  2. 删除文件并提交:执行 git rm docker/Dockerfile.nightly_torch,删除 326 行内容,提交变更。
    此外,提供了替代方案说明:用户可通过 docker build -f docker/Dockerfile --build-arg PYTORCH_NIGHTLY=1 . 实现相同的 nightly 构建效果。
文件 模块 状态 重要度
docker/Dockerfile.nightly_torch Docker 配置 removed 5.7

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

评论区精华

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

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

风险与影响

风险极低。该文件已无任何内部或外部引用,删除后不会影响现有 CI 构建流程。唯一可能受影响的是手动使用该文件构建的用户,但已提供了明确的替代方案(使用 PYTORCH_NIGHTLY=1)。如出现回归,只需恢复该文件即可,操作成本低。

对用户:若之前依赖 Dockerfile.nightly_torch 手动构建,需改用 PYTORCH_NIGHTLY=1 方式,但这类用户极少。对系统:降低仓库冗余,减少维护成本。影响范围限于 Docker 构建流程。

低风险清理 无回归风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论