执行摘要
- 一句话:修正 Ascend CI 步骤命名并禁用检查点保存
- 推荐动作:该 PR 属于低风险基础设施维护,建议快速合入。对于关注 CI 效率和可观测性的团队,值得了解 SAVE_FREQ=-1 在夜间任务中的使用方式。无需深度精读。
功能与动机
PR body 明确指出两个目的:
- 为 nightly_ascend.yml 工作流中的步骤使用正确的名称;
- 不在夜间 CI 中保存检查点。这属于持续优化 CI 基础设施的常规维护,旨在提高 CI 的可读性和资源利用效率。
实现拆解
本 PR 仅修改了 .github/workflows/nightly_ascend.yml,具体分为两部分:
-
禁用检查点保存:在 4 个 quick_start 测试步骤(quick_start_qwen3_0_6b_fsdp2_vllm_ascend、quick_start_qwen3_0_6b_megatron_vllm_ascend、quick_start_qwen3_0_6b_megatron_sglang_ascend、quick_start_qwen3_0_6b_fsdp2_sglang_ascend)的执行命令前添加了 SAVE_FREQ=-1 环境变量。这样设置后,训练脚本会跳过检查点保存,从而减少夜间 CI 的磁盘占用和运行时间。
-
修正步骤命名:将 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 配置;类别 infra;类型 infrastructure): 这是唯一变更的文件,包含步骤命名修正和禁用检查点保存的环境变量设置。
关键符号:未识别
评论区精华
该 PR 无 review 评论或讨论线程,仅有审核者 wucong25 的 APPROVED 状态,无具体评论内容。
风险与影响
- 风险:本次变更风险极低,但存在以下潜在风险:
- 若 SAVE_FREQ=-1 在某些算法或环境下不被支持,可能导致训练脚本报错,但根据现有代码逻辑,该变量是标准的检查点频率控制,风险很小。
- 步骤命名更改不会影响功能,仅影响 CI 日志的可读性。
- 该变更未引入任何源码或测试的改动,回归风险几乎为零。
- 影响:影响范围仅限于夜间 Ascend CI 工作流,对用户和系统功能无直接影响。影响程度为轻微正向:减少 CI 运行时间和磁盘占用,提升 CI 日志可读性,有助于团队更清晰地追踪 GSPO 任务的执行状态。
- 风险标记:CI 配置变更
关联脉络
- PR #7456 [ci] chore: Update ascend ci image: 同修改 nightly_ascend.yml 的 Ascend 夜间 CI 脚本,本 PR 在此基础上进一步调整步骤命名和检查点行为。
- PR #7541 [ci] chore: use Ascend recipe baselines for NPU nightly CI: 同涉及 nightly_ascend.yml 的 CI 基线配置,本 PR 的步骤命名修正与之配套演进。
参与讨论