Prhub

#47477 [ROCm][CI] Adding metadata

原始 PR 作者 AndreasKaratzas 合并时间 2026-07-03 08:45 文件变更 3 提交数 1 评论 0 代码增减 +62 / -0

执行摘要

ROCm CI 新增 Docker 构建元数据测试作业

需要在 AMD CI 中对 ROCm 镜像的 Docker 构建元数据进行测试,确保 CI base 内容哈希标签和元数据输入覆盖正确。

无需精读,属于常规 CI 基础设施改进,设计清晰、实现简洁。

讨论亮点

无 review 讨论。

实现拆解

  1. 在 .buildkite/test-amd.yaml 的 MI250 池中添加名为 "Docker Build Metadata (ROCm)" 的新 CI job,配置 source_file_dependencies 和 commands。
  2. 在 docker/Dockerfile.rocm 中新增多个 COPY 指令,将 Dockerfile、bake 脚本等文件复制到镜像中,使测试可访问这些文件。
  3. 在 tests/tools/test_docker_build_metadata_args.py 中新增两个测试函数:test_rocm_ci_base_bake_embeds_content_hash_label 和 test_rocm_ci_base_metadata_inputs_cover_ci_base_files。
文件 模块 状态 重要度
.buildkite/test-amd.yaml CI 配置 modified 4.18
docker/Dockerfile.rocm Docker 镜像 modified 3.29
tests/tools/test_docker_build_metadata_args.py 构建元数据 modified 5.63

关键符号

test_rocm_ci_base_bake_embeds_content_hash_label test_rocm_ci_base_metadata_inputs_cover_ci_base_files

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

评论区精华

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

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

风险与影响

低风险。仅涉及 CI 配置和测试文件,不影响核心运行时逻辑。但需确保新 COPY 指令不会破坏 Docker 构建缓存。

对用户无影响。对 AMD CI 系统增加了一个用于验证 ROCm Docker 构建元数据的新作业,提升 CI 覆盖率和可靠性。

CI 配置变更 Docker 构建文件复制

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论