Prhub

#6514 [hardware] chore: remove redundant Dockerfile.rocm7

原始 PR 作者 Vivicai1005 合并时间 2026-05-28 13:20 文件变更 1 提交数 1 评论 1 代码增减 +0 / -151

执行摘要

删除冗余的 ROCm 7 Dockerfile

Dockerfile.rocm7 已被 Dockerfile.rocm 取代,后者作为标准的 ROCm 7 镜像构建文件。删除冗余文件可减少维护负担,防止误用。PR body 明确说明 'This file has been superseded by Dockerfile.rocm' 并提供了 grep 验证结果。

该 PR 属于常规清理工作,无需特别关注。值得学习的是删除文件前应确保无引用。

讨论亮点

无实质性讨论。机器人审核评论无反馈,仓库管理员 wuxibin89 直接批准。

实现拆解

  1. 验证 Dockerfile.rocm7 在仓库中无其他引用(通过 grep 确认)。
  2. 直接删除 docker/rocm/Dockerfile.rocm7 文件(共 151 行)。
  3. 提交并推送,无其他代码或配置改动。
    该 PR 不涉及测试、配置或部署配套改动。
文件 模块 状态 重要度
docker/rocm/Dockerfile.rocm7 Docker 镜像 removed 4.86

关键源码片段

docker/rocm/Dockerfile.rocm7 deletion

本 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 baseARG 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}

评论区精华

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

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

风险与影响

极低风险。被删除文件已由 Dockerfile.rocm 完全取代,且仓库内无任何引用或依赖。不会影响现有构建流程或运行时行为。

影响范围仅限于仓库整洁度。对用户无影响,对系统无影响,对团队开发无负面影响。清理后避免混淆,降低维护成本。

清理工作 低风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论