Prhub

#7111 [doc] chore: update Ascend 950 references and disable nightly test

原始 PR 作者 wangdongleix 合并时间 2026-07-22 16:26 文件变更 4 提交数 1 评论 2 代码增减 +47 / -47

执行摘要

更新 Ascend 950 命名并禁用 nightly CI

PR body 说明:Update Ascend 950 product naming across related documentation and disable the Ascend Qwen3.5 35B Megatron vLLM nightly CI job。目的是保持文档与产品线实际命名一致,以及暂时禁用已知不稳定的 nightly 测试。

这是一个简单的维护 PR,可直接合并。建议后续跟踪该 nightly 测试的稳定性,并在合适时机重新启用,避免长期失去测试覆盖。

讨论亮点

没有实质性 review 讨论,PR 由维护者 wucong25 直接审核批准。

实现拆解

  1. 文档命名更新:修改 docs/ascend_tutorial/dev_guide/model_dev/transfer_to_npu_guide.mdverl/checkpoint_engine/README.mddocs/ascend_tutorial/get_start/install_guidance.rst 三处,将“A5”替换为“Ascend 950 系列产品”或“Ascend 950”,使产品命名与官方一致。
  2. CI 任务禁用:在 .github/workflows/nightly_ascend.yml 中将 nightlyCI_grpo-qwen3_5-35b-megatron-vllm_ascend 整个 job 块注释掉(包括条件、运行环境、步骤等),该任务不再执行。未改动其他 CI job 或逻辑。
  3. 配套调整:PR 只涉及文档和 CI 配置,未修改任何可执行代码,因此无需新增测试或变更 API。
文件 模块 状态 重要度
.github/workflows/nightly_ascend.yml CI 配置 modified 4.3
docs/ascend_tutorial/dev_guide/model_dev/transfer_to_npu_guide.md 文档 modified 1.53
verl/checkpoint_engine/README.md 文档 modified 1.53
docs/ascend_tutorial/get_start/install_guidance.rst 文档 modified 1.32

关键源码片段

.github/workflows/nightly_ascend.yml infrastructure

注释掉了一个 nightly CI job,直接影响测试覆盖和 CI 资源消耗。

# 以下是注释掉的 nightly CI job 起始部分(原 649-695 行整体注释):
# nightlyCI_grpo-qwen3_5-35b-megatron-vllm_ascend:
# if: ${{ github.repository_owner == 'verl-project'}}
# runs-on: linux-aarch64-a3-16
# timeout-minutes: 180
# container:
# image: swr.ap-southeast-1.myhuaweicloud.com/...:latest-cann9.0.0-...
# ...
# 该 job 被整体注释后不再触发,其余 nightly 任务不受影响。

评论区精华

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

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

风险与影响

风险极低。注释掉 nightly CI 任务可能导致该测试场景覆盖缺失,若后续恢复不及时可能遗漏回归问题。文档命名更新仅涉及文本替换,不影响功能。但注释掉的 CI 配置可能会在后续合并冲突中造成隐患(若其他人同时修改该文件)。

  • 用户/文档读者:文档中的产品名称更准确,减少混淆。
  • CI 系统:减少一个 nightly 测试任务,节省 CI 资源,但可能降低对 Qwen3.5-35B Megatron vLLM 组合的测试覆盖。
  • 团队:需要知晓该任务已被禁用,未来若问题修复应取消注释恢复测试。
CI job disabled 文档命名变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论