已合并
[docs] 补充 Mooncake池化后端 A5 环境变量与网络挂载说明 #616
[docs] 补充 Mooncake池化后端 A5 环境变量与网络挂载说明 #616
已合并
高鹏创建于 17 天前
高鹏
高鹏成员
17 天前

1. 合入背景

当前mooncake池化后端必须使用ipourma网卡,pod默认不挂载该设备,需要文档暂时给出规避方案

Fix part of #365

2. 修改内容

  1. 补充A2/A3/A5场景下,需要设置的环境变量,对齐vllm-ascend

  2. 新增ipourma网卡挂载指导。

3. 资料变更

涉及

4. 接口变更

不涉及

5. 测试结果

服务推理正常
image.png

6. CheckList

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

[x] 代码注释完备

[x] 正确记录维测日志

[x] 是否有UT用例

[x] 若涉及多线程场景,考虑了并发场景,不存在死锁问题

likedislike
Pull Request已成功合入, 合并人@wangyang
(感谢 高鹏 的贡献)
atomgit-bot
atomgit-bot
17 天前 评论:

变更摘要

本次变更仅针对 docs/zh/user_guide/features/kv_cache_store/backend/mooncake.md 文档进行补充,新增了 158 行内容,主要围绕 Mooncake 后端在昇腾 A2/A3/A5 环境下的环境变量说明及 A5 网络挂载操作指引,使其与 vllm-ascend KV Pool 的配置对齐。

主要改动

  • A2/A3/A5 环境变量说明:补充了 Mooncake 后端在昇腾平台(A2、A3、A5)上需要配置的环境变量清单,与 vllm-ascend KV Pool 的要求保持一致,便于用户在不同环境下正确配置 KV Cache 存储后端。
  • A5 网络要求文档:新增了 A5 环境下须使用 host 网络的说明,并提供了将宿主机 ipourma 网卡挂入及移出 Pod 的完整操作步骤,解决 A5 场景下的网络连通性问题。
likedislike
atomgit-bot
atomgit-bot
17 天前 评论:

代码审查

现在对已报告的位置做最终排查,确认是否有共址的次生缺陷。回顾第 98-99 行(PID 获取)和第 164-165 行(回退 sysctl),没有发现同一位置存在额外的独立缺陷。再扫描新增表格(第 35-40 行)中的环境变量值——JSON 格式正确,链接目标合理。审查完毕。


审查总结

  • 已审查文件docs/zh/user_guide/features/kv_cache_store/backend/mooncake.md(+158 行)
  • 发现问题
    • P2:1 个 — 缺少 $PID 空值校验,导致后续 netns 操作静默失败
    • P3:2 个 — 备份文件固定路径存在并发覆盖与重启丢失风险;回退脚本无条件启用宿主机 IPv6
  • 整体风险评估低风险。此变更纯为文档补充,不涉及任何运行时代码、配置逻辑或 API 契约变更。P2 问题仅影响手动运维脚本的健壮性(缺少错误处理),不会导致生产服务直接受损;两个 P3 为边界场景下的操作指引改进建议。该文档可以合并,建议后续迭代中修复 P2 的 $PID 校验问题。
类型 数量
🔴 阻塞 0
🟡 建议 1

💬 仅评论

likedislike
ascend-robotascend-robot成员
17 天前 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
17 天前 评论:

CLA Signature Pass

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

likedislike
ascend-robot
ascend-robot成员
17 天前 评论:

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/MindIE-Motor 吕有辉, 杨安 (2/2) 吕有辉 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了139条消息 查看更多
jason lyu
jason lyu成员
16 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
16 天前 添加了label:approvedlgtm
高鹏高鹏成员
16 天前 解决了最后一个问题
wangyangwangyang成员
16 天前 关闭了关联的issue
wangyangwangyang成员
16 天前 合入了pull request