Prhub

#30496 [AMD] Gate stage-c on stage-b-test-1-gpu-large-amd

原始 PR 作者 yctseng0211 合并时间 2026-07-08 15:41 文件变更 1 提交数 1 评论 1 代码增减 +1 / -0

执行摘要

修复 AMD CI 流水线 stage-c 门控缺失

PR body 明确指出:wait-for-stage-b-amd 只监控了 stage-b-test-1-gpu-small-amdstage-b-test-2-gpu-large-amd,而 stage-b-test-1-gpu-large-amd 缺失,导致该任务失败时 gate 不会快速失败,stage-c 仍然调度,浪费多 GPU 运行器。

值得合并,逻辑清晰,改动小且经过 NVIDIA 工作流验证。可快速合入。

讨论亮点

无 review 讨论。作者在 PR body 中说明了设计选择:本次仅添加 stage-b-test-1-gpu-large-amd,其他 stage-b 任务(如 -nondeterministic-mi35x 等)有意留作后续处理。

实现拆解

.github/workflows/pr-test-amd.yml 文件中,wait-for-stage-b-amd 任务的 jobs 列表中添加了一项:

  1. 新增 {"prefix": "stage-b-test-1-gpu-large-amd", "expected_count": 3},对应矩阵 part: [0,1,2],与其他 stage-b 任务格式一致。
文件 模块 状态 重要度
.github/workflows/pr-test-amd.yml CI 配置 modified 2.24

关键源码片段

.github/workflows/pr-test-amd.yml infrastructure

唯一修改的文件,在 `wait-for-stage-b-amd` 中新增 `stage-b-test-1-gpu-large-amd` 任务监控。

# .github/workflows/pr-test-amd.yml 中的 wait-for-stage-b-amd 任务
jobs: |
  [
    {"prefix": "stage-b-test-1-gpu-small-amd", "expected_count": 14},
    # 新增:监控 1-gpu-large 任务,预期 3 个(对应 part: [0,1,2])
    {"prefix": "stage-b-test-1-gpu-large-amd", "expected_count": 3},
    {"prefix": "stage-b-test-2-gpu-large-amd",   "expected_count": 2}
  ]
max-wait-minutes: '480'

评论区精华

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

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

风险与影响

风险极低:仅涉及 CI 工作流配置,不影响任何模型、内核或业务逻辑。若 expected_count 配置错误(如与实际矩阵维度不匹配),可能导致门控等待超时或误判,但 3 与现有矩阵 part: [0,1,2] 一致,风险可接受。

直接影响 AMD PR 测试流水线的 stage-c 触发时机,避免因 stage-b-test-1-gpu-large-amd 失败导致的资源浪费。对用户无影响,对开发团队而言,提高了 CI 资源利用率和反馈速度。

CI 配置变更 低风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论