执行摘要
- 一句话:更新 Ascend Docker 镜像命名规范与 CI 触发
- 推荐动作:该 PR 为简单的 CI 配置和文档更新,变更透明且经过批准,可以直接合并。开发者应了解新的镜像命名规范,并在未来版本升级时更新对应的标签字符串。
功能与动机
随着 Verl 0.8.0 的发布,需要统一 Ascend Docker 镜像的命名格式以便于识别版本和依赖。同时简化 CI 触发器,减少不必要的构建次数,降低维护成本。PR 标题直接说明仅为更新名称。
实现拆解
- 简化 CI 触发器:在四个
.github/workflows/docker-build-ascend-*.yml 中移除 release 事件触发,仅保留 push 和 paths 触发(sglang 变体还保留 schedule),避免每次发布时重复构建。
- 固定镜像标签:将原本根据分支名和 release 标签动态计算的标签逻辑替换为固定字符串,例如
latest-cann9.0.0-torch_npu2.9.0post2-910b-ubuntu22.04-py3.11-vllm,减少 CI 步骤复杂度。
- 统一多架构合并标签:在
docker buildx imagetools create 步骤中,将标签从 verl-{CANN}-{设备}-{OS}-{py}-{verl} 改为 {verl}-cann{CANN}-torch{torch_npu}-...-vllm 格式,与新的命名规范一致。
- 更新文档:修改
docs/ascend_tutorial/get_start/dockerfile_build_guidance.rst,更新每日构建和 release 镜像的命名格式说明,反映新的 pattern。
关键文件:
.github/workflows/docker-build-ascend-a2.yml(模块 CI配置;类别 infra;类型 infrastructure): 核心 CI 配置文件,修改了触发器事件和镜像标签生成逻辑,是最主要的基础设施变更。
.github/workflows/docker-build-ascend-a3.yml(模块 CI配置;类别 infra;类型 infrastructure): 与 a2 工作流对称的 A3 变体,修改模式相同。
.github/workflows/docker-build-ascend-sglang-a2.yml(模块 CI配置;类别 infra;类型 infrastructure): SGLang 推理后端的 A2 Docker 构建工作流,修改触发器和 tag 生成。
.github/workflows/docker-build-ascend-sglang-a3.yml(模块 CI配置;类别 infra;类型 infrastructure): SGLang 推理后端的 A3 Docker 构建工作流,修改模式同 SGLang A2。
docs/ascend_tutorial/get_start/dockerfile_build_guidance.rst(模块 文档;类别 docs;类型 documentation): 用户文档,更新了镜像命名格式说明,与 CI 变更保持一致。
关键符号:未识别
评论区精华
该 PR 没有收到人工审查评论,仅有 Gemini Code Assist bot 的自动审查(无反馈),随后由维护者 wucong25 直接批准。因此无实质性的设计讨论。
风险与影响
- 风险:主要风险在于 CI 工作流中镜像标签的硬编码可能因版本升级而需要手动更新;移除了 release 事件触发器,可能影响 release 时的自动构建流程。但由于这些 CI 文件仅用于 Ascend 架构的 Docker 构建,且日常使用频率不高,风险可控。文档变更可能使用户参考旧格式导致混淆,需注意同步。
- 影响:对用户:文档中镜像名称格式的更新保证了用户参考时的准确性,避免使用旧名称。对系统:CI 构建流程简化且标签标准化,后续维护更清晰。对团队:未来版本升级时需要同步更新 CI 中的标签字符串和文档说明,需纳入 checklist。
- 风险标记:CI 配置硬编码标签, 移除 release 触发器
关联脉络
- PR #6983 [doc] feat: update installation instructions for Atlas 950DT A5: 同为 Ascend 文档更新,涉及安装指导,与本 PR 的文档部分有重叠的维护上下文。
- PR #7002 [doc] feat: update installation instructions for Atlas 950DT A5: 同为 Ascend 文档更新,与本 PR 的文档部分属于同一波基础设施文档改进。
- PR #6966 [doc] feat: add installation instructions for Atlas 950DT A5: 添加 A5 安装指南,与本 PR 的文档更新均围绕 Ascend 平台体验优化。
参与讨论