执行摘要
- 一句话:临时禁用 B300 CI 任务
- 推荐动作:该 PR 简单且属于临时措施,建议快速合入以恢复 CI 稳定性,但应跟踪 #35607 并在修复后及时恢复 B300 作业。对于开发者,可关注 B300 硬件上的测试覆盖是否重要,必要时安排替代测试。
功能与动机
该 PR 旨在临时禁用 B300 GPU CI 作业,可能是因为该硬件当前不可用或存在已知问题导致 CI 频繁失败,需要暂时跳过以减少干扰。PR 标题与正文未提供详细动机,但评论“Fixed, needs #35607 instead”表明此 PR 是临时措施,可以预期后续通过 #35607 修复后重新启用。
实现拆解
- 在
.github/workflows/pr-test.yml 中,将 base-c-test-8-gpu-b300 作业定义整体注释,包括其 needs、if 和 with 字段,以避免该作业被触发。
- 在
final stage 的所需作业列表中,也将 base-c-test-8-gpu-b300 注释掉,防止因缺失该作业而误判整个 CI 为成功或失败(此处省略可能导致错误地认为成功)。
- 该改动保持注释代码,便于后续恢复,并确保 CI 流程在禁用期间仍能正确运行。
关键文件:
.github/workflows/pr-test.yml(模块 CI 配置;类别 infra;类型 infrastructure): 该文件是唯一变更,注释掉 B300 作业并更新最终阶段列表,直接影响 CI 执行。
关键符号:未识别
关键源码片段
.github/workflows/pr-test.yml
该文件是唯一变更,注释掉 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,
评论区精华
评论者 mmangkad 指出该 PR 暂时禁用后需要先合入 #35607 才能解决问题。Fridge003 批准了该 PR,未提出其他意见。
- 该 PR 需要 #35607 才能修复 (question): 该 PR 是临时措施,待 #35607 合入后再恢复。
风险与影响
- 风险:该变更禁用了 B300 相关测试,可能导致 B300 硬件上的回归未被捕获。由于 CI 阶段列表中将该作业注释掉,如果后续需要恢复,需确保该作业的依赖和条件正确。此外,禁用作业可能影响 CI 的必要测试覆盖,导致回归风险上升。
- 影响:影响范围限于 CI 基础设施,仅影响 B300 GPU 上的测试执行,不影响运行时功能。降低了 CI 失败率,但减少了 B300 硬件上的测试覆盖,可能延迟相关问题发现。
- 风险标记:CI 覆盖减少, 临时注释,需后续恢复
关联脉络
- PR #35607 unknown: 根据评论,该 PR 需要合入 #35607 后才算真正修复,因此 #35607 是后续恢复 B300 作业的关键依赖。
参与讨论