Closed
[需求]: 支持MemoryMetrics资源内存CE统计属性 #392
LuQianliAdd on Jul 16Closed at Aug 5
openUBMC-robot
Jul 16 Comments：
Jul 16 Comments：
Jul 16 Added label: sig/sig-interface
Jul 16 Assign zhangbaocun as responsible person
openUBMC-robot
Jul 16 Comments：
Jul 16 Comments：
欢迎来到 openUBMC 社区
Hey @LuQianli , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用，可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。
联系指引
有疑问可以联系 SIG: sig-interface ,
维护者是： @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是： @L_Ling, @yelvsub .
Jul 16 associated Kanban: openUBMC 26.09 需求看板
openUBMC-robot
Jul 16 Comments：
Jul 16 Comments：
Issue 工作流介绍
社区的需求和缺陷的工作流状态处于纳管中。
| 类型 | 上个状态 | 下个状态 | 评论命令 | 谁可以操作 |
|---|---|---|---|---|
| 需求 | 待办的 | 已接纳 | /todo2accepted | SIG组: sig-release-management (maintainer) |
| 需求 | 已接纳 | 开发中 | /accepted2developing | 仓库成员角色: Maintainer, Committer, Contributor, Owner |
| 需求 | 开发中 | 转测中 | /developing2testing | 仓库成员角色: Maintainer, Committer, Contributor, Owner |
| 需求 | 转测中 | 待验收 | /testing2UAT | 仓库成员角色: Maintainer, Committer, Contributor, Owner |
| 需求 | 待验收 | 已完成 | /UAT2done | SIG组: sig-QA (maintainer,committer) |
| 需求 | 待办的 | 已挂起 | /todo2suspended | SIG组: sig-release-management (maintainer) |
| 需求 | 已挂起 | 已接纳 | /suspended2accepted | SIG组: sig-release-management (maintainer) |
| 需求 | 待办的 | 已拒绝 | /todo2rejected | SIG组: sig-release-management (maintainer) |
| 需求 | 已拒绝 | 已接纳 | /rejected2accepted | SIG组: sig-release-management (maintainer) |
Jul 16 Associated pull request
Jul 16 Assign LuQianli as responsible person
Jul 16 Removed responsible person zhangbaocun
LuQianli
Jul 16 Comments：
Jul 16 Comments：
/todo2accepted
openUBMC-robot
Jul 16 Comments：
Jul 16 Comments：
Jul 16 Modify description of issue
Jul 16 Modify description of issue
Jul 16 changed custom state from 待办的 to 已接纳
Jul 16 Modify description of issue
zhaotianli
Jul 17 Comments：
Jul 17 Comments：
PR #634 回顾分析: 【8793a4361ad5a745d9487c9946580c80】新增内存CE统计属性
仓库: openUBMC/mdb_interface
分支: main → main（LuQianli/mdb_interface → openUBMC/mdb_interface）
作者: @LuQianli
合并时间: 2026-07-16 21:26:14（合并人 openUBMC-robot）
关联设计: 【待评审】支持MemoryMetrics资源内存CE统计属性
1. AI 检视意见追踪
本 PR 有两路 AI/自动化检视意见：GitCode 原生 AI 代码审查（gitcode-bot）与设计一致性检视（一致性检视机器人）。
| # | AI 来源 | 发现的问题 | 优先级 | 后续处理 | 说明 |
|---|---|---|---|---|---|
| 1 | gitcode-bot（代码审查） | CHANGELOG 新增条目版本号 [1.110.16] 与已有条目重复，且与 mds/service.json 实际版本 1.110.17 不一致 |
P2 / 建议 | ✅ 已采纳修复 | 开发者在 commit 4e4c7c9a 中修改该行，diff_comment 状态 resolved=true，最终 CHANGELOG 已为 [1.110.17] |
| 2 | 一致性检视 | CurrentPeriodCorrectableECCErrorCount/LifeTimeCorrectableECCErrorCount 缺少 options.emitsChangedSignal: "false"（设计要求变化通知=false，默认 true 与设计相反） |
严重 | ✅ 已采纳修复 | 最终代码两个属性均已包含 "emitsChangedSignal": "false"（在一致性检视 20:26 之后的最终 commit 21:18 中补齐） |
| 3 | 一致性检视 | 上述两个属性缺少 volatile: true（设计要求易变属性=true，评审讨论中亦特别强调） |
中等 | ✅ 已采纳修复 | 最终代码两个属性均已包含 "volatile": true（同上在最终 commit 中补齐） |
| 指标 | 结果 |
|---|---|
| AI 意见总数 | 3 |
| 已采纳并修复 | 3 |
| 已回复说明 | 0 |
| 未响应 | 0 |
注：一致性检视最初在 baseType/readOnly/description 上判定为一致，仅在 volatile 与 emitsChangedSignal 两处发现偏差。开发者均已在合入前修复，最终合入代码与设计属性评审表（变化通知=false、易变属性=true）完全对齐。
2. 人工检视补充
- 检视人: @yelvsub（huangleisheng）、@gb286907986（guanbin）
- 意见概要:
- yelvsub：评论
/lgtm/approve，给出检视通过并触发合入所需标签（lgtm + approved）。 - gb286907986：评论
/lgtm，但无该标签权限（机器人提示无权添加）。
- yelvsub：评论
- 解决状态: 不涉及（均为批准类操作，无 diff 行级意见需解决）
人工检视者本次未提出补充性代码意见，主要贡献是完成合入审批流程。所有代码层面问题由 AI 检视发现并被开发者采纳修复。
3. 评审贴覆盖情况
评审贴链接：【待评审】支持MemoryMetrics资源内存CE统计属性（评审结论：线下审核通过，总评审点数 2）
| 评审点 | 是否涵盖 | 说明 |
|---|---|---|
评审点1：变更资源协作接口 bmc.kepler.Systems.FDMDomain.MemoryRAS，新增 CurrentPeriodCorrectableECCErrorCount、LifeTimeCorrectableECCErrorCount |
✅ 已涵盖 | MemoryRAS.json 新增两属性，baseType/readOnly/volatile/emitsChangedSignal/description 均与设计一致 |
评审点2：Redfish MemoryMetrics 资源新增属性（/redfish/v1/Systems/{id}/Memory/{id}/MemoryMetrics） |
❌ 不在本仓范围 | 属北向 Redfish 接口，不在 mdb_interface 仓库实现范围内，需在对应 Redfish 仓库实现 |
覆盖统计: 1/2（评审点1 已涵盖；评审点2 为本仓库范围外的 Redfish 资源，符合预期分工）
4. 总结
- AI 检视采纳率 100%（3/3）：gitcode-bot 发现的 CHANGELOG 版本号问题与一致性检视发现的
emitsChangedSignal/volatile缺失问题，开发者均在合入前修复，最终代码与设计属性评审表完全对齐。 - 开发者响应积极：LuQianli 对每一条意见都通过代码修改予以落实，无任何未响应项，质量闭环良好。
- 一致性检视价值突出：本次最有价值的检视意见来自设计一致性检视（
emitsChangedSignal/volatile偏差），它补齐了原生代码审查未覆盖的设计符合性问题，是本 PR 从"部分一致"收敛到"完全一致"的关键。 - 人工检视以审批为主：两位人工检视者侧重合入审批，未补充代码意见；对于此类接口属性新增的小型变更，AI 检视已较为充分。
- 改进建议：可考虑在 CHANGELOG 与版本号同步、易变属性/变化通知等高频遗漏点上加强提交前自查，以进一步减少检视来回。
此回顾分析由 AI（OpenClaw + GLM-5.1）完成
shaohengkang
Jul 27 Comments：
Jul 27 Comments：
/accepted2developing
Jul 27 changed custom state from 已接纳 to 开发中
shaohengkang
Jul 27 Comments：
Jul 27 Comments：
/developing2testing
Jul 27 changed custom state from 开发中 to 转测中
shaohengkang
Jul 27 Comments：
Jul 27 Comments：
/testing2UAT
Jul 27 changed custom state from 转测中 to 待验收
Aug 5 changed custom state from 待验收 to 已完成
Aug 5 Close issue
Aug 11 Modify description of issue
感谢您为社区贡献新需求！请按照以下提示提供详细信息，以便我们更好地理解并跟进。
版本分支
openUBMC main
需求属性
所属版本
openUBMC26.09
工作量
5人/天
是否需要编码
是
是否免测试
否
周边依赖
不涉及
需求来源
需求描述
简要说明
支持MemoryMetrics资源内存CE统计属性；满足IPMI、Redfish RAS相关规范；支持原始数据查询，支撑伙伴/客户实现自规划差异化特性。
系统用户
运维人员、网管
前置条件
BMC运行正常
主成功场景
资源协作接口 bmc.kepler.Systems.FDMDomain.MemoryRAS 新增内存CE统计属性，Redfish MemoryMetrics 资源通过 CurrentPeriod 和 LifeTime 块提供这些统计信息，支持运维人员/网管工具查询内存本次启动周期的Correctable ECC错误统计。
扩展场景（含异常场景）
约束
不涉及
非功能需求