已合并
优化冒烟耗时 #1468
cui_jiahao创建于 18 天前
优化冒烟耗时 #1468
已合并
cui_jiahao创建于 18 天前
cui_jiahao成员
18 天前

描述

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 cui_jiahao 的贡献)
Ccui_jiahao成员
18 天前 创建了 pull request,commit 0021b265
atomgit-bot
atomgit-bot
18 天前 评论:

变更摘要

本 PR 主要围绕冒烟测试耗时优化,将原本单次的 A2 冒烟流程拆分为两个并行分片(PreSmoke_A2_part1PreSmoke_A2_part2)。在 tests/run_st.sh 中通过新增的 ST_PART 环境变量对用例进行分组控制,使两个分片分别执行不同子集;流水线中取消了原先通过 manifest-management-pluginDEPLOY/CRUSH 部署方式,改为在容器内直接调用 pre_smoke.sh 并传入 ST_PART,同时注释掉 CodeCheck 任务及 pre_smoke.sh 中的目录创建步骤以缩短流程耗时。

主要改动

  • 冒烟用例按 ST_PART 分片执行: 在 tests/run_st.sh 中为每个 python3 tests/script/run_st.py 用例前后新增 if [ -z "$ST_PART" ] || [ "$ST_PART" = "1"/"2" ] 判断,将 A3 用例划分为两部分,未设置 ST_PART 时仍执行全部用例。

  • 流水线拆分为两个并行 PreSmoke 任务: 将 PreSmoke_A2 改为 PreSmoke_A2_part1PreSmoke_A2_part2 两个 job,分别设置 export ST_PART=1export ST_PART=2 后调用 pre_smoke.sh,以并行缩短冒烟耗时。

  • 改用容器内直接执行冒烟脚本: 两个 PreSmoke job 移除了 manifest-management-pluginDEPLOY/CRUSH 及 tar 打包、wget 下载等步骤,改为在容器内通过 bash pre_smoke.sh 直接运行,并保留 obs-upload 上传 slog.tar.gz 日志。

  • 调整运行环境与资源标签: PreSmoke job 的 runs-on 标签由 ['self-hosted','arch=arm','smoke=204'] 调整为 ['self-hosted','npu=1','smoke=204'],并新增 container.image 指向 pre_smoke 镜像。

  • 注释跳过部分耗时步骤: 在 PR-pipeline_pto-isa.yml 中注释掉 JOB_codecheck 任务,并在 .gitcode/scripts/pre_smoke.sh 中注释掉 mkdir -p /home/taskspace && cd /home/taskspace,减少冒烟流程中的额外耗时环节。

likedislike
atomgit-bot
atomgit-bot
18 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
18 天前 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
18 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
18 天前 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
.gitcode/** Larry_liuyi, csjlchen, tanghaoran7 (3/2) Larry_liuyi (1/1)
repo-cann/pto-isa zhangjian_hz11, csjlchen (2/2) zhangjian_hz11 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

cui_jiahao, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
18 天前 将mywaaagh_admin,songchangxia,tanghaoran7,zhangjian_hz11,csjlchen,shentukangjun,bluesky901,changxianyu,Zhanghaijian,Larry_liuyi,syfeng,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为评审人
CANN-robotCANN-robot成员
18 天前 将mywaaagh_admin,songchangxia,tanghaoran7,zhangjian_hz11,csjlchen,shentukangjun,bluesky901,changxianyu,Zhanghaijian,Larry_liuyi,syfeng,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为审查人
cui_jiahao成员
18 天前 评论:

compile

likedislike
Ccui_jiahao成员
18 天前 update merge request[project id: 8827531, iid: 1468, commit_id: 749f4c71727824df036140ca498d5d63bb267b1f] virtual merging success
CANN-robotCANN-robot成员
18 天前 添加了label:ci-pipeline-running
zhangjian_hz11成员
18 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
18 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
18 天前 添加了label:ci-pipeline-passed
Larry成员18 天前进行代码检视2
.gitcode/workflows/PR-pipeline_pto-isa.yml
已过期
@@ -183,3 +180,1 @@
183- with:
184- precommit_image_version: ${{ jobs.JOB_image.outputs.precommit }}
185- codecheck_image_version: ${{ jobs.JOB_image.outputs.codecheck }}
180+ # JOB_codecheck:
Larry18 天前评论:

无效代码

likedislike
System
系统消息系统
18 天前 评论:

changed this line on 6b1ad068 view diff detail

Ccui_jiahao成员
18 天前 update merge request[project id: 8827531, iid: 1468, commit_id: 4fcb301f37241c39ebbd25504796a02ab2d3a5a8] virtual merging success
Ccui_jiahao成员
18 天前 强制推送  1 个提交:6b1ad068-优化冒烟耗时
Ccui_jiahao成员
18 天前 update merge request[project id: 8827531, iid: 1468, commit_id: 97fb1302c5f7a9402f10146388516382a00c53e9] virtual merging success
CANN-robotCANN-robot成员
18 天前 删除了label:cann-cla/yes
CANN-robotCANN-robot成员
18 天前 删除了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
18 天前 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
CANN-robotCANN-robot成员
18 天前 添加了label:cann-cla/yes
cui_jiahao成员
18 天前 评论:

compile

likedislike
Ccui_jiahao成员
18 天前 update merge request[project id: 8827531, iid: 1468, commit_id: 9f618072a7082587d25a29c4e0f2012f639c81d5] virtual merging success
CANN-robotCANN-robot成员
18 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
18 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
18 天前 添加了label:ci-pipeline-passed
LLarry成员
17 天前 解决了最后一个问题
Larry成员
17 天前 评论:

/approve
/lgtm

likedislike
zhangjian_hz11成员
17 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
17 天前 添加了label:approved
tanghaoran
tanghaoran成员
17 天前 评论:

/lgtm

likedislike
csjlchen
csjlchen成员
17 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
17 天前 添加了label:lgtm
CANN-robotCANN-robot成员
17 天前 合入了pull request