已开启
fix: 支持SSU定位灯控制(Redfish/Web) #1505
huangminghao创建于 8 天前
fix: 支持SSU定位灯控制(Redfish/Web) #1505
已开启
合并受阻
8 天前 关联了issue：PowerConverter接口Schema文件补充字段定义
8 天前 将WeliveWelove,DataSculptor,huanghan,aaronlee设为审查人
8 天前 添加了label：sig/sig-interface
openUBMC-robot
8 天前 评论：
8 天前 评论：
欢迎来到 openUBMC 社区
Hey @yelmh , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用，可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则，更多详情，请参阅机器人自助配置使用手册。
联系指引
有疑问可以联系 SIG: sig-interface ,
维护者是： @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是： @L_Ling, @PLUS_ONE, @yelvsub .
8 天前 添加了label：ai-co-authored
8 天前 添加了label：openUBMC-cla/yes
8 天前 添加了label：ai-includes-code
openUBMC-robot
8 天前 评论：
8 天前 评论：
8 天前 添加了label：needs-issue
openUBMC-robot
8 天前 评论：
8 天前 评论：
关联 Issue 通知
@yelmh , 提交的PR需要关联issue，指导链接：https://forum.openubmc.cn/t/topic/208
如果已经关联了issue, 但是needs-issue标签仍然存在，你可以在评论 /check-issue 来移除它。
8 天前 添加了label：ai-compliance-successful
openUBMC-robot
8 天前 评论：
8 天前 评论：
The pipeline(pipeline number:2059) is running. >>> Check Pipeline <<<.Please wait a moment...
8 天前 添加了label：ci-running
openUBMC-robot
8 天前 评论：
8 天前 评论：
/ai review
atomgit-bot
8 天前 评论：
8 天前 评论：
8 天前 删除了label：ci-running
openUBMC-robot
8 天前 评论：
8 天前 评论：
| 任务名称 | 结果 |
|---|---|
| pre-commit检查 | ✅SUCCESS |
| 防投毒检查 | ✅SUCCESS |
| 开源合规检查 | ✅SUCCESS |
| 编译构建 | ✅SUCCESS |
| ipmi修改 | ✅SUCCESS |
| 修改行数检查 | ✅SUCCESS |
| AI Review | ✅SUCCESS |
8 天前 添加了label：ci-successful
8 天前 关联了issue：[需求]: 支持ASU模组定位灯控制
interface_config/redfish/mapping_config/Chassis/Chassis.json
@@ -1389,6 +1389,179 @@
13891389 }
13901390 ]
13911391 },
1392+ {
zhaotianli
2 天前 评论：
2 天前 评论：
PR #1505 Code Review: fix: 支持SSU定位灯控制(Redfish/Web)
仓库: openUBMC/rackmount
分支: feat/ssu-uid-led → main
关联设计: topic 7191（SSU Redfish北向，已评审）、topic 7410（Web/CLI管理SSU，已评审）——本 PR 仅定位灯部分
变更概览
| 文件 | 新增行 | 说明 |
|---|---|---|
| redfish/mapping_config/Chassis/Chassis.json | +173 | 新增 /redfish/v1/Chassis/SSU:id GET/PATCH |
| web_backend/mapping_config/System/SSUs.json | +168 | 新增 /UI/Rest/System/SSUs 列表与 :id GET/PATCH |
| redfish/plugins/orchestrator/chassis.lua | +34 | get_ssu_path / get_ssu_led_path |
| web_backend/plugins/orchestrator/chassis.lua | +42 | get_ssu_path / get_ssu_led_path / get_ssu_list |
| schemastore .../dmtf/json_schema/chassis.json | +2/-1 | uris 新增 /redfish/v1/Chassis/SSU:id |
逐项 Review
| 维度 | 结果 | 说明 |
|---|---|---|
| D14 路径/@odata | ✅ | URI、@odata.type(#Chassis.v1_26_0.Chassis)、@odata.id、ChassisType=Module 与设计一致；SSU:id 模板与既有 Expansion:id 模式一致；schemastore uris 已同步（仅 en 目录，无遗漏） |
| D15 属性一致性 | ❌ | Web 列表 List 元素结构与设计不符（见问题1）；详情接口电子标签属性未实现（PR 声明后续交付，见提示2） |
| D16 关联结构 | ✅ | ChassisCollection Members 经既有 get_chassis_members()（遍历 /bmc/dev 下 bmc.dev.Chassis）自动纳入 SSU，与设计 Members 示例一致 |
| D17 操作与语义 | ⚠️ | GET/PATCH 齐全、ReqBody boolean、ResourceExist 404 符合设计；但 PATCH Interval 时长语义与 PR 描述矛盾（见问题3） |
| D19 权限 | ⚠️ | LockdownAllow: true 与既有定位灯实现不一致（见问题4） |
| D22 数组索引 | ✅ | ProcessingFlow 确认为 1-based（与既有 GetUIDPath 引用 ProcessingFlow[3] 模式核对一致）。Redfish GET 3 条流引用 [1]/Destination/Id、[2]/Destination/Name、[3]/Destination/State 及 Web GET 单条流引用 [1]/Destination/State 均指向正确步骤；新增资源为独立数组块，不影响既有索引 |
| D23 数据映射 | ✅ | GET↔PATCH 互逆：true↔5、false↔0，255→true 为历史常亮值兜底，default→null 符合设计 boolean,null；Switch case 全覆盖 |
| D24 接口一致性 | ❌ | 调用模式与 get_expansion_path/主机定位灯 SetState(State, Interval) + CallIf #WITH 完全一致；但 bmc.dev.Chassis/bmc.dev.Board 需先在 mdb_interface 定义（见问题2） |
| D25 通用 | ⚠️ | JSON 语法 CI 通过；CHANGELOG.md 未更新（见问题5） |
发现的问题
1.（major，D15/D17）Web SSU 列表 List 元素结构与设计不一致
- 位置：web_backend/plugins/orchestrator/chassis.lua get_ssu_list（约 122-138 行）+ SSUs.json:9
- 设计帖 7410 评审点1 明确
List元素为对象：[{ "SSU": "/UI/Rest/System/SSUs/SSU0" }]，而实现返回纯字符串数组["/UI/Rest/System/SSUs/SSU0", ...]，Web 前端按item.SSU取值会得到 undefined。 - 建议：plugin 中
table.insert(list, { SSU = path })（参考 utils.lua get_fru_list 直接构造对象数组的先例），或在 mapping 中增加 L-Pair 包装步骤。
2.（major，D24）资源协作接口须先在 mdb_interface 社区接口库定义
- 位置：Chassis.json:1462（bmc.dev.Chassis）、Chassis.json:1470 附近（bmc.dev.Board）
- 已有检视意见（DataSculptor，未解决）指出 bmc.dev.Chassis 在 mdb_interface 中未定义。补充：GET ProcessingFlow[2] 使用的 bmc.dev.Board 在本仓零先例使用，同样需确认已在 mdb_interface 定义/申报；否则编译或运行期接口校验会失败。
- 建议：先合入 mdb_interface 接口定义（或在 PR 中说明定义所在仓/PR 链接），并核对 ssu_card.sr 南向是否同时实现 bmc.dev.Chassis（Id）与 bmc.dev.Board（Name）。
3.（major，D17）PATCH Interval=255 的时长语义与 PR 描述矛盾
- 位置：Chassis.json PATCH SetState Params（约 1548-1552 行）、SSUs.json 同构
- chassis 仓 identify 倒计时为 interval×10 秒，255 → 2550 秒（42.5 分钟），而 PR 描述与测试建议写的是"闪烁 255 秒自动灭"（已有检视意见指出）。另 PR 描述"false→SetState(0, 0)"与实现 (0, 255) 不符（State=0 时 Interval 无功能影响，但文档应与实现一致）。
- 建议：与需求方确认预期定位时长后修正 PR 描述/测试建议，或调整 Interval 取值；顺带统一 false 分支描述。
4.（⚠️，D19）LockdownAllow: true 与既有定位灯实现不一致
- 位置：Chassis.json SSU PATCH（约 1504 行）、SSUs.json PATCH
- 主机 Chassis 与 Drives 的 LocationIndicatorActive PATCH 均未设置 LockdownAllow（即锁定模式下拒绝），本 PR SSU 定位灯允许锁定模式操作，且设计帖未声明 lockdown 行为。请确认该差异为有意设计。
5.（minor，D25）CHANGELOG.md 未更新
- 仓库惯例每个功能 PR 均追加条目，本 PR 未更新。
提示项（不阻塞）
- Web GET /UI/Rest/System/SSUs/:id 缺设计帖的 Manufacturer/SerialNumber/Model/PartNumber（电子标签），Redfish GET 同缺电子标签/PowerState/Status/Drives/NetworkAdapters/Links——PR 已声明仅定位灯部分，请在后续 PR 补齐并在 PR 描述列出交付拆分。
- SystemId 硬编码为 1，多主机机型是否适用请确认。
- Redfish GET 边界：SSU Chassis 存在但 Led 对象缺失时（GetSSULedPath 返回空串），ProcessingFlow[3] 的预期行为（返回 null vs 报错）建议补充自验证。
总结
整体结构遵循了仓库既有模式（Expansion:id 路由、GetPath 插件、SetState+CallIf #WITH、1-based 索引、GET↔PATCH 互逆映射），Redfish 侧实现质量较好。综合判定：部分一致——需优先解决 Web 列表结构不符（问题1）与 mdb_interface 接口定义（问题2，已有未决检视意见），并澄清定位时长语义（问题3）后再合入。
⏱️ 本 PR 检视耗时：430.0 秒（约 7.2 分钟）
interface-sig评审贴：
（仅定位灯部分）
当前PR是否有AI参与:
[x] 是
__1. Agent平台信息: Trae
__2. 模型信息: Claude
__3. Prompt摘要: 按两份接口评审贴定位灯结论，结合component_drivers仓ssu_card.sr南向契约（Led_SSUUID为CPLD UID灯位直控，仅支持闪烁/关闭）与chassis仓SetState/主从控制实现，调整SSU定位灯Redfish/Web北向映射
PR功能描述 / 为什么需要这个合入:
SSU模组定位灯经Redfish/Web北向控制：南向为bmc.kepler.Systems.Led的SetState（chassis仓实现，含主从控制）。SSU定位灯硬件仅支持闪烁/关闭，不支持常亮。
修改方案:
验证方案:
测试建议:
自验证报告:
该PR关联的issue:
fixes #
希望检视人员了解: