执行摘要
- 一句话:删除冗余的 ROCm 7 Dockerfile
- 推荐动作:该 PR 属于常规清理工作,无需特别关注。值得学习的是删除文件前应确保无引用。
功能与动机
Dockerfile.rocm7 已被 Dockerfile.rocm 取代,后者作为标准的 ROCm 7 镜像构建文件。删除冗余文件可减少维护负担,防止误用。PR body 明确说明 'This file has been superseded by Dockerfile.rocm' 并提供了 grep 验证结果。
实现拆解
- 验证
Dockerfile.rocm7 在仓库中无其他引用(通过 grep 确认)。
- 直接删除
docker/rocm/Dockerfile.rocm7 文件(共 151 行)。
- 提交并推送,无其他代码或配置改动。
该 PR 不涉及测试、配置或部署配套改动。
关键文件:
docker/rocm/Dockerfile.rocm7(模块 Docker 镜像;类别 infra;类型 deletion): 本 PR 唯一变更文件,删除冗余 Dockerfile,清理仓库。
关键符号:未识别
关键源码片段
docker/rocm/Dockerfile.rocm7
本 PR 唯一变更文件,删除冗余 Dockerfile,清理仓库。
# 该文件已删除,由 Dockerfile.rocm 替代
# 原文件内容如下(仅展示头部):
# default base image
ARG REMOTE_VLLM="1"
ARG COMMON_WORKDIR=/app
ARG BASE_IMAGE=rocm/vllm-dev:base
FROM ${BASE_IMAGE} AS base
ARG ARG_PYTORCH_ROCM_ARCH
ENV PYTORCH_ROCM_ARCH=${ARG_PYTORCH_ROCM_ARCH:-${PYTORCH_ROCM_ARCH}}
# Install some basic utilities
RUN apt-get update -q -y && apt-get install -q -y \
sqlite3 libsqlite3-dev libfmt-dev libmsgpack-dev libsuitesparse-dev \
apt-transport-https ca-certificates wget curl
RUN python3 -m pip install --upgrade pip
RUN apt-get purge -y sccache; python3 -m pip uninstall -y sccache; rm -f "$(which sccache)"
ARG COMMON_WORKDIR
WORKDIR ${COMMON_WORKDIR}
评论区精华
无实质性讨论。机器人审核评论无反馈,仓库管理员 wuxibin89 直接批准。
风险与影响
- 风险:极低风险。被删除文件已由
Dockerfile.rocm 完全取代,且仓库内无任何引用或依赖。不会影响现有构建流程或运行时行为。
- 影响:影响范围仅限于仓库整洁度。对用户无影响,对系统无影响,对团队开发无负面影响。清理后避免混淆,降低维护成本。
- 风险标记:清理工作, 低风险
关联脉络
- PR #6466 [ci] chore: update npu docker to cann 9.0.0: 同为 Docker 相关清理与更新,体现团队对容器镜像的持续维护。
参与讨论