oam-tools/.gitcode · starchen_/oam-tools

cann-robotci(asys): 摘除 pre-commit python 钩子屏蔽并修复存量告警

文件	最后提交记录	最后更新时间
ISSUE_TEMPLATE	Initial commit	8 个月前
scripts	ci(asys): 摘除 pre-commit python 钩子屏蔽并修复存量告警 Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !552 merge ci/asys-precommit-lint into master ci(asys): 摘除 pre-commit python 钩子屏蔽并修复存量告警 Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述将 src/asys 与 test/{ut,st}/asys 从 ruff、pylint、bandit、incremental-codecheck 的 exclude 中摘除，使这些目录纳入 pre-commit 门禁，并修复因此暴露的全部存量告警。复刻已落地的 msaicerr 范式（commit 2e60c0a）：摘除即真实修复、src 零内联屏蔽。 ## 变更类型请选择本次引入的变更类型（勾选对应项）： - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新（格式化，局部变量） - [x] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [x] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 变更内容 ### 配置侧 - 四钩子（ruff/pylint/incremental/bandit）src 块从 `src/(asys\|msaicerr)/` 删 asys → `src/(msaicerr)/`（msaicerr 保留，不在本 PR 范围） - 四钩子 test 行从 `\| test/` 改为 `\| test/(?!(ut\|st)/asys/)`（负向前瞻，只摘 asys 测试，其他测试目录保留屏蔽） ### 代码侧修复（src/asys，零 noqa/disable/nosec） - ruff: F401 重导出用 `__all__`（53 项，不删公共 API）；F541/F841 真实修复；PLR6301×8 `@staticmethod`；T201 spinner 改 `sys.stdout.write`（保 `\r` 覆写） - pylint: W0718 broad-except 收窄为具体异常（ctypes→`(OSError, AttributeError, TypeError)`；文件 I/O→`OSError`；二进制解析→`(OSError, TypeError, IndexError, KeyError)`）；W1514 open 补 `encoding='utf-8'`；W0612/W0613 未用变量/参数；W0622 重定义内建 `dir`→`dir_path`；W0603 `global` 改可变容器；W0212 `getattr`；W0125 常量条件；W1510 `subprocess.run` 补 `check=False`；W1509 `preexec_fn`→`start_new_session=True`；W0123 `eval`→`getattr`；W0246 删无用 `__init__` 委托；W1309/W0611 修生成源 `.in`/`.cmake` 模板 - bandit: B602 `shell=True`→`[BASH, "-c", cmd]` + `shell=False`（BASH=shutil.which）；B605 `os.popen`→`subprocess.run` arg list；B307 `eval`→`getattr` ### 安全加固（src/asys） - `cmd_run.py`: `check_command` 改 `shutil.which`；`run_command`/`run_cmd_output`/`run_linux_cmd`/`real_time_output`/`popen_run_cmd` 去 `shell=True` 改 `[BASH, "-c", cmd]` + `check=False` + `try/except` 保返回语义 - `asys_launch.py`: `Popen(shell=True)` → `[BASH, "-c", cmd]` + `start_new_session=True` - `asys_profiling.py`: `subprocess.run(shell=True)` → `[BASH, "-c", cmd]` - `ops_collect.py`: `os.popen(grep)`×3 → `subprocess.run([GREP, args])` - `param_dict.py`: `eval(f"args.{arg_name}")` → `getattr(args, arg_name)` ### 测试侧修复（test/{ut,st}/asys） - ruff: E402 sys.path/sys.argv 收敛到现有 conftest；F401/F541/F841/F811/E722 修复 - pylint: 文件级 disable pytest 固有写法（protected-access/unused-argument/broad-exception-caught 等）；W1514/W1510 真实修复 - bandit: B103 chmod 测试断言用 `# nosec` 注明；B108 `/tmp`→`tempfile.gettempdir()`；B602 测试 mock 用 `# nosec` 注明 - codespell: finsh/Unknow/utiliza/flie 等拼写修复 ### 暴露并修复的真实缺陷 - F811 重复测试 `test_asys_config_get_mode`（同名函数互相遮蔽，前者从未执行）；重命名影子测试后发现其在 VM 上失败（config 命令不可执行），skip 并注明 ### 生成文件根因（chip_handler.py，gitignored） - `.in` 模板删未用 `import os`/`log_error`；`ascend910B.cmake` `rf"910B\d"`→`r"910B\d"`（去掉多余 f 前缀） ## 关联的 Issue 无（独立 PR，参照 msaicerr commit 2e60c0a 范式） ## 如何测试 `bash bash build.sh -u --component asys # 本地验证：ruff/pylint/bandit/incremental-codecheck/codespell 五钩子全通过 # UT: asys 596 passed、msaicerr 723 passed/15 skipped（rebase 后含 master 新增用例） # ST 281 passed` > 本地未覆盖 msprof gtest（需编译），依赖云端 `UT_Test`。 ## 核对清单 - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签（如：`feat:`, `fix:`） - [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md） ## 其他信息 - 已 rebase 到 master 070f767（PR #533 msaicerr 摘除已合入）。冲突仅 `.pre-commit-config.yaml`，两侧意图正交、按叠加解决：`src/` 屏蔽块两组件均已摘除故整块删除；`test/` 负向前瞻合并为 `test/(?!(ut\|st)/(asys\|msaicerr)/)`；保留 msaicerr 侧 `test/(ut\|st)/msaicerr/res/` 例外（asys 测试树的 `data/` 无 Python 文件，无需对应例外）。非配置文件的改动逐字节一致。 - rebase 后 master 带入的 asys 文件（`src/asys/view/table.py`、`test/ut/asys/testcase/view/test_table.py`、`test_run_tests_parsing.py`）此前被 exclude 屏蔽，摘除后已实跑五钩子确认通过。 - 负例参数化 `'Unknow'` → `'Unknown'`（3 文件 6 处）：回退到 upstream 原值后被 codespell 判为 typo，而这些文件在本 PR 改动列表内、codespell 钩子无 `exclude` 按整文件扫描，会卡云端 precommit。`get_supported_chip_info` 仅做 `910B\d`/`910_93`/`910_96`/`950` 正则匹配，`Unknown` 同样不匹配、仍返回 `False`，负例语义不变。 - src/asys 零内联屏蔽（`grep -rnE 'noqa\|pylint: disable\|nosec' src/asys` 为空，不含 gitignored 的 chip_handler.py） - 保留 msaicerr 于各 exclude（msaicerr 由其自己的 PR 摘除） - 本 PR 与 hccl_test/msprof PR 代码改动零重叠（仅 .pre-commit-config.yaml 共享，语义不重叠） See merge request: cann/oam-tools!552	2 天前
workflows	ci: add package tests as UT component Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !562 merge feat/ut-package into master ci: add package tests as UT component Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: 新增 UT_package 组件，执行安装、卸载和升级 ST；包测试日志直接输出到流水线终端。 See merge request: cann/oam-tools!562	2 天前
PULL_REQUEST_TEMPLATE.en-US.md	【文档翻译】部分文档翻译成英文。 Co-authored-by: lwx1255555<liuxiaofang17@huawei-partners.com> # message auto-generated for no-merge-commit merge: !238 merge master into master 【文档翻译】部分文档翻译成英文。 Created-by: lwx1255555 Commit-by: lwx1255555 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> md文档翻译成英文 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> 不涉及 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> 已经自检 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> 更新的文档如下。 .claude/CLAUDE_en.md .claude/skills/default-skills/SKILL_en.md .gitcode/PULL_REQUEST_TEMPLATE.en-US.md .opencode/README_en.md AGENTS_en.md CONTRIBUTING_en.md README_en.md SECURITY_en.md docs/en/quick_install.md src/hccl_test/README_en.md ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [x] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/oam-tools!238	3 个月前
PULL_REQUEST_TEMPLATE.zh-CN.md	更新ReadMe文档，Quick_install文档，Pull Template Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !143 merge dev_master_updateoamtools into master 更新ReadMe文档，Quick_install文档，Pull Template Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 更新README.md：环境安装部署使用quick_install.md的章节，安装oam-tools新增grep版本高于3.8时的告警提醒更新quick_install.md：依赖组件新增patch，coverage，pytest-mock 更新.gitcode/PULL_REQUEST_TEMPLATE.zh-CN.md：新增类型标签 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> 关联Issue [#52](https://gitcode.com/cann/oam-tools/issues/52) 关联Issue [#46](https://gitcode.com/cann/oam-tools/issues/46) 关联Issue [#53](https://gitcode.com/cann/oam-tools/issues/53) ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> 文档更新，不涉及 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> 更新了README.md文件更新了quick_install.md文件 ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [x] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/oam-tools!143	5 个月前