已合并
[fix]日志内容整改 #4988
[fix]日志内容整改 #4988
已合并
devHaven创建于 15 天前
devHaven成员
15 天前

描述

本次主要是日志的整改,主要涉及语法错误、拼写错误、级别错配、错误描述不完整、未知缩写、冗余日志、禁止中文、魔鬼数字等。所有修改仅涉及日志文案和格式字符串,未改变任何业务逻辑、变量名或控制流。

变更类型

请选择本次引入的变更类型:

关联的Issue

https://gitcode.com/cann/hcomm/issues/812

测试

不涉及新增UT用例,不修改逻辑,因此不需要checker和真机验证

文档更新

不涉及

合入检查

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 devHaven 的贡献)
DdevHaven成员
15 天前 创建了 pull request,commit 7b8527b9
atomgit-bot
atomgit-bot
15 天前 评论:

变更摘要

本 PR(#4988,标题 [fix])是一次纯日志整改,范围覆盖集群链路诊断(experimental/cluster_link_diag)、src/base_commsrc/legacy 下 910/950 平台的 HCCL 通信与资源管理模块。全部改动仅涉及日志文案与格式字符串,未改变任何业务逻辑、变量名或控制流。整改内容主要包括:修正日志级别错配(将大量 HCCL_INFO/log_info 的失败与告警场景提升为 HCCL_WARNING/log_warning)、修正拼写与语法错误(如 peerankpeerRankromateremotedo not availableis not availablemust greater thanmust be greater than 等)、将禁止的中文日志改为英文、补全错误描述缺失的参数(消除魔鬼数字)以及修正日志中引用错误的变量名。

主要改动

  • 日志级别错配修正:在 experimental/cluster_link_diag/dispatcher/dispatcher_common.py 新增 log_warning 并在 ssh_push_controller.py 中将三处 [warning] 日志由 log_info 改为 log_warninghccl_nslbdp.ccadapter_hccp.ccccu_ins_preprocessor.cppmc2_compont.cpp 等将 strncpy_s/memcpy_s 失败、socket 收发失败、资源分配失败等场景由 HCCL_INFO/HCCL_RUN_INFO 提升为 HCCL_WARNING/HCCL_RUN_WARNING
  • 拼写错误修正:批量修正错误单词,包括 peerankpeerRankall_reduce_recursive_hd.cc)、romateremotedev_rdma_connection_v2.cchost_rdma_connection.cc)、conndIdconnIdbkf_ch_cli_letcp.c)、reducescaterReduceScatterrecommandedrecommendedcreakLinkcreateLinkTransportQuerryTransportQueryunavaialbleunavailablelcoalSizelocalSizeSignalInexSignalIndexkernalLaunchkernelLaunchintputPtrinputPtrerrornoerrnodevideIddeviceId 等。
  • 英文语法规范化:统一修正日志英文表达,如 do not availableis not availabledo not emptyis not emptymust greater thanmust be greater thanmust less thanmust be less thanis not supportis not supportedalready exitalready existsno existdoes not existcannot foundcannot findformfromwaitimewaittimeoccuredoccurred 等。
  • 中文日志转英文与错误描述补全parallel_command_executor.py"错误捕获输出" 改为 "error captured output"probe_controller.cpp"无法打开输出文件!" 改为 "failed to open output file!";同时补全缺失的参数与描述,如 SetDeterministicConfig 日志补充 deterministic[%u]interface_hccl.ccwqePerDoorbell 非法日志补充实际值与 MAX_WQE_PER_DOORBELL 上限、aicpu_kfc_rpc_serverv2.cc 补充 HCCL_MSG_CNTall_reduce_chunk_mesh.cc 补全缺失的 round 格式参数。
  • 修正日志引用的错误标识符:修正引用错误或拼错的常量/变量名,包括 CCU_SQ_UBFFER_SIZECCU_SQ_BUFFER_SIZEccu_jetty_ctx_mgr.ccccu_jetty_ctx_mgr.cpp)、ctxHanldectxHandlelegcylegacynormal_docket_lonormal_docker_loctxSignautrectxSignature;并修正日志中实际打印的错误变量,如 hccl_socket_manager.cc 的 client socket map 大小打印、sqe_mgr.ccmemcpy_s 改为实际调用的 memset_shccl_impl.ccpreempt_port_manager.cc 的时态表述等。
likedislike
不准确?
atomgit-bot
atomgit-bot
15 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
15 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
15 天前 评论:

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
experimental 颜业峰, 杨小渊, 兰泽泽, 劳大钊 (4/2) 颜业峰 (1/1)
src/base_comm 劳大钊, 孙哲芃, 颜业峰, 杨小渊, 兰泽泽 (5/2) 孙哲芃, 颜业峰 (2/1)
src/base_comm/resources/hccp 陈轲, 颜业峰, 杨小渊, 兰泽泽, 劳大钊 (5/2) 陈轲, 颜业峰 (2/1)
src/coll_communicator_mgr 颜业峰, 丁炜秦, 杨小渊, 兰泽泽, 劳大钊 (5/2) 颜业峰, 丁炜秦 (2/1)
src/legacy/ascend910/algorithm 李标智, 颜业峰, 杨小渊, 兰泽泽, 劳大钊 (5/2) 李标智, 颜业峰 (2/1)
src/legacy/ascend910/common 孙哲芃, 颜业峰, 杨小渊, 兰泽泽, 劳大钊 (5/2) 孙哲芃, 颜业峰 (2/1)
src/legacy/ascend910/framework 颜业峰, 丁炜秦, 杨小渊, 兰泽泽, 劳大钊 (5/2) 颜业峰, 丁炜秦 (2/1)
src/legacy/ascend910/framework/device/aicpu_kfc 兰泽泽, 劳大钊, 杨小渊, 颜业峰 (4/2) 颜业峰 (1/1)
src/legacy/ascend910/hccd 兰泽泽, 劳大钊, 丁炜秦, 颜业峰, 杨小渊 (5/2) 颜业峰, 丁炜秦 (2/1)
src/legacy/ascend910/platform 孙哲芃, 颜业峰, 杨小渊, 兰泽泽, 劳大钊 (5/2) 孙哲芃, 颜业峰 (2/1)
src/legacy/ascend950/framework 颜业峰, 丁炜秦, 兰泽泽, 劳大钊, 杨小渊 (5/2) 颜业峰, 丁炜秦 (2/1)
src/legacy/ascend950/service/collective/alg 杨小渊, 李标智, 颜业峰, 兰泽泽, 劳大钊 (5/2) 李标智, 颜业峰 (2/1)
src/legacy/ascend950/unified_platform 杨小渊, 兰泽泽, 劳大钊, 颜业峰, 孙哲芃 (5/2) 颜业峰, 孙哲芃 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了90条消息 查看更多
Innoecho成员
8 天前 评论:

/approve

likedislike
bz_lee成员
8 天前 评论:

/approve

likedislike
philo
philo成员
8 天前 评论:

/approve

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