Prhub

#35627 [CI] Temporarily disable B300 jobs

原始 PR 作者 mmangkad 合并时间 2026-08-21 08:54 文件变更 1 提交数 1 评论 1 代码增减 +16 / -15

执行摘要

临时禁用 B300 CI 任务

该 PR 旨在临时禁用 B300 GPU CI 作业,可能是因为该硬件当前不可用或存在已知问题导致 CI 频繁失败,需要暂时跳过以减少干扰。PR 标题与正文未提供详细动机,但评论“Fixed, needs #35607 instead”表明此 PR 是临时措施,可以预期后续通过 #35607 修复后重新启用。

该 PR 简单且属于临时措施,建议快速合入以恢复 CI 稳定性,但应跟踪 #35607 并在修复后及时恢复 B300 作业。对于开发者,可关注 B300 硬件上的测试覆盖是否重要,必要时安排替代测试。

讨论亮点

评论者 mmangkad 指出该 PR 暂时禁用后需要先合入 #35607 才能解决问题。Fridge003 批准了该 PR,未提出其他意见。

实现拆解

  1. .github/workflows/pr-test.yml 中,将 base-c-test-8-gpu-b300 作业定义整体注释,包括其 needsifwith 字段,以避免该作业被触发。
  2. final stage 的所需作业列表中,也将 base-c-test-8-gpu-b300 注释掉,防止因缺失该作业而误判整个 CI 为成功或失败(此处省略可能导致错误地认为成功)。
  3. 该改动保持注释代码,便于后续恢复,并确保 CI 流程在禁用期间仍能正确运行。
文件 模块 状态 重要度
.github/workflows/pr-test.yml CI 配置 modified 4.34

关键源码片段

.github/workflows/pr-test.yml infrastructure

该文件是唯一变更,注释掉 B300 作业并更新最终阶段列表,直接影响 CI 执行。

# 临时禁用 B300 测试作业
# base-c-test-8-gpu-b300:
# needs: [check-changes, call-gate, wait-for-base-b, sgl-kernel-build-wheels, rust-ext-build]
# if: ${{ !failure() && !cancelled() }}
# uses: ./.github/workflows/_pr-test-stage.yml
# with:
# self_name: base-c-test-8-gpu-b300
# runner_config: 8-gpu-b300
# check_changes: ${{ toJson(needs.check-changes.outputs) }}
# caller_inputs: ${{ toJson(inputs) }}
# partitions: ${{ needs.check-changes.outputs.partitions }}
# run_timeout_minutes: '60'
# timeout_per_file: '3600'
# rust_ext_artifact: ${{ needs.rust-ext-build.outputs.artifact_name }}
# secrets: inherit# 在最终阶段列表中移除 B300 作业,防止 CI 状态计算错误
# base-c-test-8-gpu-b300,

评论区精华

该 PR 需要 #35607 才能修复 question

mmangkad 评论指出该 PR 需要 #35607 才能修复。

结论:该 PR 是临时措施,待 #35607 合入后再恢复。 · 已解决

风险与影响

该变更禁用了 B300 相关测试,可能导致 B300 硬件上的回归未被捕获。由于 CI 阶段列表中将该作业注释掉,如果后续需要恢复,需确保该作业的依赖和条件正确。此外,禁用作业可能影响 CI 的必要测试覆盖,导致回归风险上升。

影响范围限于 CI 基础设施,仅影响 B300 GPU 上的测试执行,不影响运行时功能。降低了 CI 失败率,但减少了 B300 硬件上的测试覆盖,可能延迟相关问题发现。

CI 覆盖减少 临时注释,需后续恢复

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论