| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
【需求】使用DB格式减少解析后数据 Co-authored-by: Mad97_<chepishuai1@h-partners.com> # message auto-generated for no-merge-commit merge: !87 merge B030 into master 【需求】使用DB格式减少解析后数据 Created-by: Mad97_ Commit-by: Mad97_ Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因:** 增加特性: 1.原始输入数据大小超过500M,则在终端提示用户合理配置命令行参数来减少解析时长和数据落盘量 2.当format参数不含json时,算子目录解析也不包含json - **修改内容:** - [ ] **涉及代码双合**(贴上另一个PR链接): ---- ### 2. 功能验证 - [Y] **功能自验** - [ ] **本地自验用例截图**(请确保不体现个人信息) - [ ] **冒烟是否通过** ---- ### 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [Y] **是否经过代码检视** - [Y ] **是否具备UT测试用例看护** ---- ### 4. 安全自检 **典型安全编码问题** - [Y] **若涉及对外接口,是否已校验外部数据** - [Y] **MR 标题和描述是否按格式填写** - [NA] **是否进行空指针校验** - [Y] **是否进行返回值校验** - [NA] **是否正确考虑文件权限配置** - [NA] **是否充分考虑接口的异常场景** - [Y] **是否正确记录错误日志** - [NA] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** - [NA] **若涉及运算,是否存在整数溢出、除零等风险** ---- ### 5. 变更知会 - **资料修改:** NA - **变更通知(消息知会 + 邮件知会):** NA ---- ### 6. 冒烟修改 - **PR 来源:** - [ ] 问题单 - [Y] 需求特性 - [ ] 安全排查 - [ ] 其他 - [NA] **是否存在冒烟可以拦截却未拦截的情况** - [Y] **是否需要添加冒烟:** ---- See merge request: Ascend/msserviceprofiler!87 | 7 个月前 | |
【msserviceprofiler】vllm采集解析前冒烟开发 Co-authored-by: hewenbo<hewenbo16@h-partners.com> # message auto-generated for no-merge-commit merge: !214 merge master into master 【msserviceprofiler】vllm采集解析前冒烟开发 Created-by: hewenbo Commit-by: hewenbo Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因:【msserviceprofiler】vllm采集解析前冒烟开发 - **修改内容:【msserviceprofiler】vllm采集解析前冒烟开发 - [ ] **涉及代码双合**(贴上另一个PR链接): ---- ### 2. 功能验证 - [ ] **功能自验** - [ ] **本地自验用例截图**(请确保不体现个人信息) - [ ] **冒烟是否通过** ---- ### 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [ ] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** ---- ### 4. 安全自检 **典型安全编码问题** - [ ] **若涉及对外接口,是否已校验外部数据** - [ ] **MR 标题和描述是否按格式填写** - [ ] **是否进行空指针校验** - [ ] **是否进行返回值校验** - [ ] **是否正确考虑文件权限配置** - [ ] **是否充分考虑接口的异常场景** - [ ] **是否正确记录错误日志** - [ ] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** - [ ] **若涉及运算,是否存在整数溢出、除零等风险** ---- ### 5. 变更知会 - **资料修改:** - **变更通知(消息知会 + 邮件知会):** ---- ### 6. 冒烟修改 - **PR 来源:** - [ ] 问题单 - [ ] 需求特性 - [ ] 安全排查 - [ ] 其他 - [ ] **是否存在冒烟可以拦截却未拦截的情况** - [ ] **是否需要添加冒烟:** ---- See merge request: Ascend/msserviceprofiler!214 | 5 个月前 | |
【msserviceprofiler_master】SGlang在最新镜像采集解析问题修复 Co-authored-by: hewenbo<hewenbo16@h-partners.com> # message auto-generated for no-merge-commit merge: !404 merge master into master 【msserviceprofiler_master】SGlang在最新镜像采集解析问题修复 Created-by: hewenbo Commit-by: hewenbo Merged-by: ascend-robot Description: # PR 合入模板 ## 1. 修改描述 - **修改原因:【msserviceprofiler_master】SGlang采集解析问题修复 - **修改内容:【msserviceprofiler_master】SGlang采集解析问题修复 - [ ] **涉及代码双合**(贴上另一个PR链接): ---- ## 2. 功能验证 - [x] **功能自验** - [ ] **本地自验用例截图**(请确保不体现个人信息) - [ ] **冒烟是否通过** ---- ## 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [x] **是否经过代码检视** - [x] **是否具备UT测试用例看护** ---- ## 4. 安全自检 **典型安全编码问题** - [x] **若涉及对外接口,是否已校验外部数据** - [x] **MR 标题和描述是否按格式填写** - [x] **是否进行空指针校验** - [x] **是否进行返回值校验** - [x] **是否正确考虑文件权限配置** - [x] **是否充分考虑接口的异常场景** - [x] **是否正确记录错误日志** - [x] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** - [x] **若涉及运算,是否存在整数溢出、除零等风险** ---- ## 5. 变更知会 - **资料修改:** - **变更通知(消息知会 + 邮件知会):** ---- ## 6. 冒烟修改 - **PR 来源:** - [ ] 问题单 - [ ] 需求特性 - [ ] 安全排查 - [x] 其他 - [ ] **是否存在冒烟可以拦截却未拦截的情况** - [ ] **是否需要添加冒烟:** ---- See merge request: Ascend/msserviceprofiler!404 | 2 个月前 | |
【ms-service-metric】【ST】监控大盘冒烟框架首次搭建 Co-authored-by: Mad97_<chepishuai1@h-partners.com> # message auto-generated for no-merge-commit merge: !373 merge metric_st into master 【ms-service-metric】【ST】监控大盘冒烟框架首次搭建 Created-by: Mad97_ Commit-by: Mad97_ Merged-by: ascend-robot Description: # PR 合入模板 ## 1. 修改描述 - **修改原因:** ms-service-metric 当前缺少 vLLM 场景级 ST 冒烟能力,无法快速验证 vllm-ascend 不同版本下 hook 加载、metric 开关、推理请求和 /metrics 暴露链路是否正常;同时后续 EPLB/DPLB/异常场景需要可扩展的场景框架承载。 - **修改内容:** - 新增 Metric ST 统一入口 run_metric_st.py,默认执行 basic 场景。 - 新增 test/st/python/metric 场景框架: - metric_runner.py:统一处理安装 ms-service-metric、启动 vLLM、执行 ms_service_metric restart、发送请求、抓取 /metrics、保存 artifacts。 - metric_scenarios.py:按 basic/eplb/dplb/exception 注册场景,隔离启动参数、preflight 和指标契约。 - metric_assertions.py:提供通用 Prometheus 文本解析和指标断言能力。 - metric_smoke_utils.py:提供模型、NPU、端口、Prometheus 目录、metric 安装等前置检查。 - 新增 pytest 用例 test_metric_vllm_basic.py,支持通过 --metric-scenario 选择场景。 - 新增 EPLB 启动适配: - 设置 DYNAMIC_EPLB=true - 自动按设备数设置 --tensor-parallel-size - 开启 --enable-expert-parallel - 通过 --additional-config 注入 eplb_config - 启动日志需确认 Dynamic EPLB is True - 新增 artifacts 输出: - preflight.log - metric_install.json - service.log - request.log - metrics.raw - assertion_report.json - 增加 test/ut/python/test_metric_st_framework.py,覆盖 metric parser、断言规则、场景参数生成等基础逻辑。 - 增加 test/st/python/metric/README.md,说明基础和专项场景执行方式。 - [ ] **涉及代码双合**(贴上另一个PR链接): 暂无。 ---- ## 2. 功能验证 - [ ] **功能自验**  - [ ] **本地自验用例截图**(请确保不体现个人信息) - [ ] **冒烟是否通过** ---- ## 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [ ] **是否经过代码检视** 待组织检视。本次变更超过 200 行,建议按要求进行三人以上会议检视。 - [x] **是否具备UT测试用例看护** 新增 test/ut/python/test_metric_st_framework.py,覆盖 metric 文本解析、断言规则、场景注册和参数生成等框架逻辑。 ---- ## 4. 安全自检 **典型安全编码问题** - [x] **若涉及对外接口,是否已校验外部数据** CLI 参数增加 choices、请求次数校验、场景名校验;模型路径、NPU、端口、metric source 均有前置检查。 - [x] **MR 标题和描述是否按格式填写** 已按模板填写。 - [x] **是否进行空指针校验** Python 代码中对可选参数、缺失配置文件、缺失模块、缺失命令等情况进行了分支处理。 - [x] **是否进行返回值校验** 对 git/pip/vllm help/ms_service_metric restart/curl 等命令返回值进行了检查。 - [x] **是否正确考虑文件权限配置** Prometheus multiproc 目录创建后会进行写入探测;artifacts 写入固定在 workspace 下。 - [x] **是否充分考虑接口的异常场景** 覆盖模型不存在、NPU 不可用、NPU 占用过高、端口冲突、metric 安装失败、vLLM 启动失败、指标缺失等异常。 - [x] **是否正确记录错误日志** 失败时保留 preflight、service、request、metrics 和 assertion report。 - [x] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** Prometheus sample 解析正则仅面向单行文本,不包含高风险嵌套回溯结构。 - [x] **若涉及运算,是否存在整数溢出、除零等风险** 仅涉及端口、设备数、请求次数等简单整数处理;请求次数已校验 >= 1。 ---- ## 5. 变更知会 - **资料修改:** 新增 test/st/python/metric/README.md,说明 Metric ST 执行方式和场景选择方式。 - **变更通知(消息知会 + 邮件知会):** 待 PR 提交后按团队流程知会。 ---- ## 6. 冒烟修改 - **PR 来源:** - [ ] 问题单 - [x] 需求特性 - [ ] 安全排查 - [ ] 其他 - [ ] **是否存在冒烟可以拦截却未拦截的情况** 当前 PR 目标是补齐 ms-service-metric 的基础 ST 框架,历史上缺少对应冒烟拦截能力。 - [x] **是否需要添加冒烟:** 是。本 PR 新增 Metric ST 基础框架和 basic/eplb/dplb/exception 场景入口;本次先合入框架和基础校验,后续 PR 继续扩展 EPLB/DPLB/异常场景的字段覆盖。 See merge request: Ascend/msserviceprofiler!373 | 3 个月前 | |
【ms-service-metric】【BugFix】修复vLLM全局指标phase标签污染 Co-authored-by: Mad97_<chepishuai1@h-partners.com> # message auto-generated for no-merge-commit merge: !379 merge phase_fix into master 【ms-service-metric】【BugFix】修复vLLM全局指标phase标签污染 Created-by: Mad97_ Commit-by: Mad97_ Merged-by: ascend-robot Description: # PR 合入模板 ## 1. 修改描述 - **修改原因:** vLLM 指标新增 phase 标签后,部分本不应按 phase 区分的全局指标被全局 phase 状态影响,导致同一指标被拆分为 decode/mixed/prefill/unknown 多条时间序列。例如 kvcache 总数、空闲数、已分配数等指标出现多条 phase 曲线,影响 PromQL 聚合和前台展示。 - **修改内容:** 1. 调整 vLLM 指标 phase 处理逻辑:不应区分 phase 的全局/状态类指标固定导出为 phase="all"。 2. 需要区分 phase 的执行链路指标继续保留动态 phase,例如 decode/mixed/prefill。 3. 修复 kvcache、scheduler、batch size、memory 等指标的 phase 归类。 4. 补充 UT 用例,覆盖全局指标固定 phase="all"、动态指标保留 phase 等场景。 5. 调整 metric ST 框架中指标采集 restart 时机,在 vLLM 服务启动前开启采集,避免启动期一次性指标未被采集。 6. 补充 ST phase 检查,验证全局指标不会被拆成多条 phase 曲线,动态指标仍保留预期 phase。 - [ ] **涉及代码双合**(贴上另一个PR链接): 不涉及 ---- ## 2. 功能验证 - [x] **功能自验** 已执行 metric ST phase 检查,结果通过: dynamic metric phases OK: vllm_profiling_executor:model_runner_execute_model:duration_count -> ['decode', 'mixed', 'prefill'] dynamic metric phases OK: vllm_profiling_record_function_or_nullcontext_count -> ['decode', 'mixed', 'prefill'] PHASE CHECK PASSED, checked=13 已验证纳入检查的 13 类指标符合预期:全局指标固定为 phase="all",动态执行链路指标保留 decode/mixed/prefill,kvcache 指标不再被拆分为多条 phase 曲线。 - [ ] **本地自验用例截图**(请确保不体现个人信息)   - [x] **冒烟是否通过** 已通过 metric ST 冒烟,phase 检查通过,未发现 fatal/warning 异常。 ---- ## 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [ ] **是否经过代码检视** 待检视 - [x] **是否具备UT测试用例看护** 已补充/调整 vLLM metric/meta handler 相关 UT,以及 metric ST 框架 restart 时机相关 UT。 ---- ## 4. 安全自检 **典型安全编码问题** - [ ] **若涉及对外接口,是否已校验外部数据** 不涉及对外接口变更 - [x] **MR 标题和描述是否按格式填写** - [ ] **是否进行空指针校验** 不涉及新增空指针风险 - [ ] **是否进行返回值校验** 不涉及新增外部调用返回值处理逻辑 - [ ] **是否正确考虑文件权限配置** 不涉及文件权限配置变更 - [x] **是否充分考虑接口的异常场景** 已考虑指标未触发、hook 不存在、不同 phase 类型指标的兼容场景 - [ ] **是否正确记录错误日志** 不涉及新增错误日志 - [ ] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** 不涉及正则表达式变更 - [ ] **若涉及运算,是否存在整数溢出、除零等风险** 不涉及新增整数运算风险 ---- ## 5. 变更知会 - **资料修改:** 不涉及用户使用方式变更;如需同步前台展示或 PromQL 样例,可后续更新示例配置。 - **变更通知(消息知会 + 邮件知会):** 需知会前台/看板使用方:部分全局指标的 phase 将统一为 all,PromQL 中如依赖 phase=~"decode|prefill|mixed" 查询全局指标,需要调整为 phase="all" 或按指标类型区分查询。 ---- ## 6. 冒烟修改 - **PR 来源:** - [x] 问题单 - [ ] 需求特性 - [ ] 安全排查 - [ ] 其他 - [x] **是否存在冒烟可以拦截却未拦截的情况** 存在。原冒烟未覆盖全局指标 phase 被拆线的问题。 - [x] **是否需要添加冒烟:** 需要,已补充 phase 检查逻辑,覆盖全局指标固定 phase="all"、动态指标保留 decode/mixed/prefill 的验证场景。 ``` See merge request: Ascend/msserviceprofiler!379 | 3 个月前 | |
init master Co-authored-by: msserviceprofilerDeveloper<msserviceprofilerDeveloper> # message auto-generated for no-merge-commit merge: !13 merge out into master init master Created-by: jiangruitao Commit-by: jiangruitao;eveyin1;tt0cool;ascend-robot;xzm123;AvadaKedavrua;minghang_c;ChenHuiwen;jiangruitao 00800418;msserviceprofilerDeveloper Merged-by: ascend-robot Description: init master See merge request: Ascend/msserviceprofiler!13 | 8 个月前 | |
init master Co-authored-by: msserviceprofilerDeveloper<msserviceprofilerDeveloper> # message auto-generated for no-merge-commit merge: !13 merge out into master init master Created-by: jiangruitao Commit-by: jiangruitao;eveyin1;tt0cool;ascend-robot;xzm123;AvadaKedavrua;minghang_c;ChenHuiwen;jiangruitao 00800418;msserviceprofilerDeveloper Merged-by: ascend-robot Description: init master See merge request: Ascend/msserviceprofiler!13 | 8 个月前 | |
init master Co-authored-by: msserviceprofilerDeveloper<msserviceprofilerDeveloper> # message auto-generated for no-merge-commit merge: !13 merge out into master init master Created-by: jiangruitao Commit-by: jiangruitao;eveyin1;tt0cool;ascend-robot;xzm123;AvadaKedavrua;minghang_c;ChenHuiwen;jiangruitao 00800418;msserviceprofilerDeveloper Merged-by: ascend-robot Description: init master See merge request: Ascend/msserviceprofiler!13 | 8 个月前 | |
init master Co-authored-by: msserviceprofilerDeveloper<msserviceprofilerDeveloper> # message auto-generated for no-merge-commit merge: !13 merge out into master init master Created-by: jiangruitao Commit-by: jiangruitao;eveyin1;tt0cool;ascend-robot;xzm123;AvadaKedavrua;minghang_c;ChenHuiwen;jiangruitao 00800418;msserviceprofilerDeveloper Merged-by: ascend-robot Description: init master See merge request: Ascend/msserviceprofiler!13 | 8 个月前 | |
【ms-service-metric】【ST】增强监控指标冒烟校验与日志健康检查 Co-authored-by: Mad97_<chepishuai1@h-partners.com> # message auto-generated for no-merge-commit merge: !377 merge metric_st_add into master 【ms-service-metric】【ST】增强监控指标冒烟校验与日志健康检查 Created-by: Mad97_ Commit-by: Mad97_ Merged-by: ascend-robot Description: # PR 合入模板 ## 1. 修改描述 - **修改原因:** 补齐 ms_service_metric 监控指标的 ST/冒烟看护能力,解决指标是否正常注册、请求后是否实际增长、服务日志中是否存在指标采集异常等问题缺少自动化拦截的风险。 - **修改内容:** - 新增监控指标冒烟框架,覆盖 basic、eplb、dplb 等场景。 - 新增 test/st/python/metric/metric_runner.py,负责启动 vLLM 服务、安装/加载 metric 包、采集 /metrics 前后快照、执行请求并保存冒烟产物。 - 新增 metric_assertions.py,支持 Prometheus 文本解析、指标存在性校验、label 校验、数值规则校验、请求后 counter 增长校验、可选/触发型指标处理。 - 新增 metric_scenarios.py,定义 basic/EPLB/DPLB 场景参数、vLLM 启动参数、场景能力预检和指标期望。 - 新增 metric_log_health.py,扫描 vLLM 服务日志中的 ms_service_metric 错误、注册失败、记录失败等异常日志,作为冒烟失败条件。 - 新增 metric_smoke_utils.py,封装模型路径、NPU、端口、Prometheus multiprocess 目录、metric 包安装、运行时信息采集等预检能力。 - 新增 run_metric_st.py 入口和 test/st/python/metric/README.md 使用说明。 - 新增 test/ut/python/test_metric_st_framework.py,对指标解析、期望匹配、增长判断、日志健康检查、场景参数生成等核心逻辑进行 UT 看护。 - [ ] **涉及代码双合**(贴上另一个PR链接): 不涉及 / 待补充 ---- ## 2. 功能验证 - [ ] **功能自验** basic python run_metric_st.py --scenario basic --model-path /data/models/Qwen3-30B-A3B-w8a8 --device 0 --metric-source /home/chepishuai/workspace_st_add/msserviceprofiler --workspace /home/chepishuai/workspace_st_add  eplb python run_metric_st.py --scenario eplb --model-path /data/models/Qwen3-30B-A3B-w8a8 --device 2 --device 3 --metric-source /home/chepishuai/workspace_st/msserviceprofiler --workspace /home/chepishuai/metric_st_workspace  - [ ] **本地自验用例截图**(请确保不体现个人信息) 待补充 - [ ] **冒烟是否通过** 待补充实际冒烟结果 ---- ## 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [ ] **是否经过代码检视** 待补充。该变更新增 ST 框架和用例,建议按大于 200 行变更组织三人以上检视。 - [x] **是否具备UT测试用例看护** 已新增 test/ut/python/test_metric_st_framework.py,覆盖 Prometheus 指标解析、断言规则、增长判断、日志健康检查和场景参数生成。 ---- ## 4. 安全自检 **典型安全编码问题** - [x] **若涉及对外接口,是否已校验外部数据** ST runner 对模型路径、NPU 设备、端口、metric source、poll 参数等做了预检或参数约束。 - [x] **MR 标题和描述是否按格式填写** - [x] **是否进行空指针校验** 对可选指标、可选 baseline、可选 report path、可选 runtime 信息等场景进行了分支处理。 - [x] **是否进行返回值校验** 对命令执行返回码、vLLM 服务启动、HTTP 健康检查、请求执行结果、指标断言结果进行了校验。 - [x] **是否正确考虑文件权限配置** 冒烟产物写入用户指定 workspace,并对 Prometheus multiprocess 目录进行创建和写入探测。 - [x] **是否充分考虑接口的异常场景** 覆盖命令不存在、模型路径不存在、NPU 不可用、端口占用、服务启动失败、请求失败、指标未增长、日志异常等失败路径。 - [x] **是否正确记录错误日志** 冒烟失败保留 preflight.log、service.log、request.log、metrics.raw、assertion_report.json、log_health_report.json 等定位产物。 - [x] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** 正则用于解析 Prometheus sample 和筛选日志关键字,表达式结构简单、无高风险嵌套回溯。 - [x] **若涉及运算,是否存在整数溢出、除零等风险** 主要为计数、超时、轮询间隔和指标数值比较;已对 request count、poll timeout、poll interval 等输入做范围校验。 ---- ## 5. 变更知会 - **资料修改:** 已新增/更新 test/st/python/metric/README.md,说明 basic/eplb/dplb 场景、执行命令、参数、断言逻辑和产物路径。 - **变更通知(消息知会 + 邮件知会):** 待补充 ---- ## 6. 冒烟修改 - **PR 来源:** - [ ] 问题单 - [x] 需求特性 - [ ] 安全排查 - [ ] 其他 - [ ] **是否存在冒烟可以拦截却未拦截的情况** 当前变更用于补齐监控指标冒烟拦截能力,未发现已知“应拦截但未拦截”的历史问题;待实际 ST 运行后补充结论。 - [x] **是否需要添加冒烟:** 是。本 PR 新增 ms_service_metric 指标冒烟框架和 basic/eplb/dplb 场景看护。 See merge request: Ascend/msserviceprofiler!377 | 3 个月前 | |
【ST】【SGLang】补充sglang冒烟用例 Co-authored-by: ChenHuiwen<chenhuiwen7@huawei.com> # message auto-generated for no-merge-commit merge: !207 merge add-ut into master 【ST】【SGLang】补充sglang冒烟用例 Created-by: ChenHuiwen Commit-by: ChenHuiwen Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因:** 【ST】【SGLang】补充sglang冒烟用例 - **修改内容:** 【ST】【SGLang】补充sglang冒烟用例,新增一个sglang基础服务化数据采集用例 - [ ] **涉及代码双合**(贴上另一个PR链接): ---- ### 2. 功能验证 - [ ] **功能自验** - [ ] **本地自验用例截图**(请确保不体现个人信息) - [x] **冒烟是否通过**   ---- ### 3. 代码检视 - **要求:** - 合入代码大于 200 行,需三人以上会议检视。 - 检视密度≥2个/100行。 - 检视缺陷密度达不到要求的需给出说明。 - 大于 1000 行代码原则上不允许合入,需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** ---- ### 4. 安全自检 **典型安全编码问题** - [x] **若涉及对外接口,是否已校验外部数据** - [x] **MR 标题和描述是否按格式填写** - [x] **是否进行空指针校验** - [x] **是否进行返回值校验** - [x] **是否正确考虑文件权限配置** - [x] **是否充分考虑接口的异常场景** - [x] **是否正确记录错误日志** - [x] **若涉及正则表达式,是否对正则表达式做 ReDos 校验** - [x] **若涉及运算,是否存在整数溢出、除零等风险** ---- ### 5. 变更知会 - **资料修改:** - **变更通知(消息知会 + 邮件知会):** ---- ### 6. 冒烟修改 - **PR 来源:** - [ ] 问题单 - [ ] 需求特性 - [ ] 安全排查 - [x] 其他 - [ ] **是否存在冒烟可以拦截却未拦截的情况** - [ ] **是否需要添加冒烟:** ---- See merge request: Ascend/msserviceprofiler!207 | 5 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 7 个月前 | ||
| 5 个月前 | ||
| 2 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 8 个月前 | ||
| 3 个月前 | ||
| 5 个月前 |