已合并
【FEAT】Simplify rank table parsing and add IPv6 support for HCCL collect #4989
刘家旺创建于 3月10日
【FEAT】Simplify rank table parsing and add IPv6 support for HCCL collect #4989
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 刘家旺 的贡献）3月10日 添加了label：stat/needs-squash
ascend-robot
3月10日 评论：
3月10日 评论：
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 |
|---|---|---|
| repo-Ascend/msit | ✅ jhon-117, minghang_c (2/2) | ✅ jhon-117 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
3月10日 添加了label：ascend-cla/yes
刘家旺
3月10日 评论：
3月10日 评论：
compile
此处折叠了72条消息 查看更多
Masaka
3月11日 评论：
3月11日 评论：
/lgtm
kai1949
3月11日 评论：
3月11日 评论：
/approve
3月11日 添加了label：approvedlgtm
ascend-robot
3月11日 评论：
3月11日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: jhon-117.
Reviewers who wrote a comment of /lgtm are: jhon-117, minghang_c.
3月11日 合入了pull request，合并节点 SHA：b93db1ec7a1ad90e5d3b0599b26d465ca28d980f
PR Type / PR类型
issue：https://gitcode.com/Ascend/msit/issues/400
问题/功能描述
本次PR主要实现了对IPv6网络环境的支持，并重构了RankTable解析模块以提升代码的复用性、可维护性和健壮性。具体解决了HCCL网络连通性检查工具在处理IPv6地址时无法正确构建命令的问题，同时通过简化解析架构、统一IP地址验证和错误处理，优化了Ascend设备相关的工具函数。
修改方案描述
IPv6支持：在hccl.py中，修改_run_cmd方法，利用ipaddress模块动态判断IP地址版本，并据此构建正确的hccn_tool命令行参数，从而实现对IPv6地址的连通性测试。
RankTable解析重构：将原有基于抽象基类和注册器的复杂解析架构，重构为基于函数字典的扁平化设计。将MindIEParser和VLLMParser类实现替换为独立的解析函数，并统一使用ipaddress模块进行IP验证，替代原有的正则表达式，同时集中定义了规范的异常类型。
接口统一与测试：在utils模块中更新导出接口，移除了旧类型，新增了统一的Framework枚举和parse_rank_table函数。此外，新增了全面的单元测试，覆盖了正常解析、异常数据处理及边界条件等场景，确保函数的鲁棒性。