Prhub

#44497 [CI] Reverted gitignore changes

原始 PR 作者 AndreasKaratzas 合并时间 2026-06-04 15:37 文件变更 3 提交数 1 评论 0 代码增减 +0 / -14

执行摘要

回退 dockerignore 改动并修复 ROCm SCCACHE 崩溃

PR body 中明确说明:需要完全回退 dockerignore 改动以解除发布阻塞,并解决 ROCm 构建中 sccache 因 SCCACHE_ENDPOINT= 空值导致崩溃的问题(InvalidUri(Empty))。

该 PR 为应急修复,技术价值有限但维护了 CI/CD 稳定性。建议阅读以了解 sccache 环境变量空值导致崩溃的潜在陷阱,以及 Docker 构建中 .dockerignore 与 git 工作树检查的冲突问题。

讨论亮点

无 review 讨论。该 PR 仅由 khluu 审核并批准,没有额外评论。

实现拆解

  1. 回退 .dockerignore 文件:移除之前添加的排除规则(docs/、.github/、.pre-commit-config.yaml、format.sh),恢复为原始内容。
  2. 回退 tools/check_repo.sh 文件:删除之前添加的 Docker 环境检测逻辑,该逻辑试图从 .git 对象数据库中恢复被排除的文件以保持工作树整洁。
  3. 修复 Dockerfile.rocm 中的 sccache 配置:移除 ENV SCCACHE_ENDPOINT=${USE_SCCACHE:+${SCCACHE_ENDPOINT}} 行,防止 sccache 因空值崩溃。该环境变量只在 USE_SCCACHE=1 时设置,但在某些场景下 ENV 会被展开为空字符串,导致 sccache 解析失败。
文件 模块 状态 重要度
tools/check_repo.sh 构建脚本 modified 3.15
.dockerignore 构建脚本 modified 2.64
docker/Dockerfile.rocm 部署脚本 modified 2.4

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

评论区精华

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

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

风险与影响

回归风险:回退 .dockerignore 和 check_repo.sh 改动后,之前试图解决的“Docker 构建中工作树不一致”问题可能重新出现,但 PR body 称这是为了解除发布阻塞的主动权衡。
构建风险:SCCACHE_ENDPOINT 删除后,若某些 ROCm 构建依赖该变量(尽管之前为空),可能需同步调整构建环境。
风险较低:变更均为文件删除,且已经过审核和合并。

影响范围:ROCm Docker 构建流程和发布流程。
影响程度:中等——修复了 sccache 崩溃,回退可能影响之前优化的工作树一致性,但优先保证构建稳定性。
用户影响:只影响 ROCm 构建者,不会影响最终用户。

回归风险 发布阻塞修复

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论