已合并
fix:whl安装移动到 install_common_parser 前面,先安装whl包再统一权限规范 #172
qinhan创建于 9 天前
fix:whl安装移动到 install_common_parser 前面,先安装whl包再统一权限规范 #172
已合并
qinhan创建于 9 天前
qinhan
qinhan
9 天前

描述

本 PR 调整了 scripts/package/ops_ras/scripts/opp_install.sh 中 OPP 安装流程的执行顺序:将 install_es_whl 和 install_cann_ops_ras_whl 两个 whl 安装调用从 add_init_py 之后移动到 install_common_parser(COMMON_PARSER_FILE 调用)之前,使 whl 包先安装、再由后续流程统一进行权限规范。同时把删除 ${TARGET_VERSION_DIR}/ops_ras 目录的逻辑从 install_es_whl 移入 add_init_py,使清理动作随初始化脚本添加环节统一执行。

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 qinhan 的贡献)
qinhanqinhan
9 天前 创建了 pull request,commit 4d4cc78c
atomgit-bot
atomgit-bot
9 天前 评论:

变更摘要

本 PR 调整了 scripts/package/ops_ras/scripts/opp_install.sh 中 OPP 安装流程的执行顺序,将 install_es_whl 和 install_cann_ops_ras_whl 两个 whl 安装调用提前到 install_common_parser(即 COMMON_PARSER_FILE 调用)之前,使 whl 包先安装、再由后续流程统一进行权限规范。同时,将删除 ${TARGET_VERSION_DIR}/ops_ras 目录的逻辑从 install_es_whl 迁移到 add_init_py,并把 whl 包安装中的权限设置动作移除。

主要改动

  • 调整 whl 安装时机: 在 install_opp 中把 install_es_whl 与 install_cann_ops_ras_whl 的调用从 add_init_py 之后移动到 bash "${COMMON_PARSER_FILE}" ... 之前,实现先安装 whl 包再统一权限规范。
  • 移除 install_whl_package 中的权限设置: 删除了对 ${_pythonlocalpath}/${_package_name} 及其 dist-info 目录执行 chmod -R "${CUSTOM_PERM}" 的两行逻辑,权限规范交由后续统一流程处理。
  • 迁移目录清理逻辑: 将 install_es_whl 中删除 ${TARGET_VERSION_DIR}/ops_ras 目录的判断与 rm -rf 操作移除,并新增到 add_init_py 中,使清理动作随初始化脚本添加环节统一执行。

🧩 本次 Review 已使用自定义 Skill:

likedislike
不准确?
atomgit-bot
atomgit-bot
9 天前 评论:

代码审查

✅ 未发现问题


🧩 本次 Review 已使用自定义 Skill:

likedislike
不准确?
CANN-robotCANN-robot成员
9 天前 添加了label:cann-cla/yes
qinhanqinhan
9 天前 修改了pull request 的描述
CANN-robot
CANN-robot成员
9 天前 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
9 天前 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
repo-cann/ops-ras ✅ yue-ma, 柳宗谷 (2/2) ✅ yue-ma (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
qinhan
qinhan
9 天前 评论:

compile

likedislike
CANN-robotCANN-robot成员
9 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
9 天前 评论:

流水线任务触发成功
任务链接 [7b1ea6644bd048d09be3504c47bc7987][流水线指导]

任务名称状态日志下载链接
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-10-16 11:01:07]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
9 天前 评论:

流水线任务触发成功
任务链接 [1df83a6a1c6d46258d1eba1058246c98][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single_ubuntu24 ✅ SUCCESS >>>>> >>>>>
UT_Test_ophost ✅ SUCCESS
UT_Test_opapi ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>

[2026-10-16 11:03:12]    CI执行结束

likedislike
CANN-robotCANN-robot成员
9 天前 添加了label:api-check-pass
CANN-robotCANN-robot成员
9 天前 删除了label:api-check-pass
CANN-robot
CANN-robot成员
9 天前 评论:

流水线任务触发成功
任务链接 [5c7bcab41c2a4b3ebf7e2e2490037188][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
codecheck_codestyle ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-10-16 11:04:08]    CI执行结束

likedislike
CANN-robotCANN-robot成员
9 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9 天前 添加了label:ci-pipeline-passed
yue-ma成员
9 天前 评论:

/lgtm
/approve

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

/lgtm

likedislike
ZZYuan
ZZYuan成员
9 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
9 天前 添加了label:lgtm
CANN-robot
CANN-robot成员
9 天前 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: SuperYuan
/lgtm: SuperYuan

likedislike
CANN-robotCANN-robot成员
9 天前 合入了pull request