Prhub

#7549 [ci] chore: correct step naming for Ascend ci

原始 PR 作者 lxb007981 合并时间 2026-08-26 09:40 文件变更 1 提交数 1 评论 0 代码增减 +6 / -6

执行摘要

修正 Ascend CI 步骤命名并禁用检查点保存

PR body 明确指出两个目的:

  1. 为 nightly_ascend.yml 工作流中的步骤使用正确的名称;
  2. 不在夜间 CI 中保存检查点。这属于持续优化 CI 基础设施的常规维护,旨在提高 CI 的可读性和资源利用效率。

该 PR 属于低风险基础设施维护,建议快速合入。对于关注 CI 效率和可观测性的团队,值得了解 SAVE_FREQ=-1 在夜间任务中的使用方式。无需深度精读。

讨论亮点

该 PR 无 review 评论或讨论线程,仅有审核者 wucong25 的 APPROVED 状态,无具体评论内容。

实现拆解

本 PR 仅修改了 .github/workflows/nightly_ascend.yml,具体分为两部分:

  1. 禁用检查点保存:在 4 个 quick_start 测试步骤(quick_start_qwen3_0_6b_fsdp2_vllm_ascendquick_start_qwen3_0_6b_megatron_vllm_ascendquick_start_qwen3_0_6b_megatron_sglang_ascendquick_start_qwen3_0_6b_fsdp2_sglang_ascend)的执行命令前添加了 SAVE_FREQ=-1 环境变量。这样设置后,训练脚本会跳过检查点保存,从而减少夜间 CI 的磁盘占用和运行时间。

  2. 修正步骤命名:将 GSPO 任务的步骤名称从 Running nightlyCI_grpo-qwen3-8b-fsdp2-vllm_ascend 更正为 Running nightlyCI_gspo-qwen3-8b-fsdp2-vllm_ascend(以及对应的 checking script 步骤),使其与执行的脚本 run_gspo_qwen3_8b_fsdp2_npu.sh 相匹配,提升 CI 日志的可读性和可追踪性。

由于变更仅涉及 CI 配置,无新增测试或文档配套,也无需改动其他模块。

文件 模块 状态 重要度
.github/workflows/nightly_ascend.yml CI 配置 modified 3.46

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

评论区精华

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

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

风险与影响

本次变更风险极低,但存在以下潜在风险:

  • 若 SAVE_FREQ=-1 在某些算法或环境下不被支持,可能导致训练脚本报错,但根据现有代码逻辑,该变量是标准的检查点频率控制,风险很小。
  • 步骤命名更改不会影响功能,仅影响 CI 日志的可读性。
  • 该变更未引入任何源码或测试的改动,回归风险几乎为零。

影响范围仅限于夜间 Ascend CI 工作流,对用户和系统功能无直接影响。影响程度为轻微正向:减少 CI 运行时间和磁盘占用,提升 CI 日志可读性,有助于团队更清晰地追踪 GSPO 任务的执行状态。

CI 配置变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论