合并受阻
欢迎来到 openUBMC 社区
Hey @byosoft_zhangchao , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用，可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则，更多详情，请参阅机器人自助配置使用手册。
联系指引
有疑问可以联系 SIG: sig-interface ,
维护者是： @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是： @L_Ling, @PLUS_ONE, @yelvsub .
关联 Issue 通知
@byosoft_zhangchao , 提交的PR需要关联issue，指导链接：https://forum.openubmc.cn/t/topic/208
如果已经关联了issue, 但是needs-issue标签仍然存在，你可以在评论 /check-issue 来移除它。
The pipeline(pipeline number:2052) is running. >>> Check Pipeline <<<.Please wait a moment...
/ai review
⚠️ 警告：Redfish 侧同名聚合逻辑未同步修复
interface_config/redfish/plugins/orchestrator/power.lua:383 中的 get_chassis_ps_component_health() 仍只匹配 component_obj.Name == 'PowerSupply'（power.lua:393）。若本平台组件名确为 PS1/PS2 格式，Redfish 侧 Chassis 电源冗余健康聚合会复现同样的问题。
修复建议：在 Redfish 侧 get_chassis_ps_component_health() 中同步采用相同匹配逻辑，保持两条接口的聚合结果一致。
💡 建议：命名匹配可考虑覆盖 PSU# 与裸 PS 格式
当前正则 ^PS%d+$ 仅覆盖 PS1/PS2 与 PowerSupply。仓库内其他插件存在 PSU<slot>（web_backend/plugins/orchestrator/upgrade.lua:65 的 '^PSU(%d+)$'、redfish/plugins/orchestrator/chassis_assembly.lua:118 的 'PSU' .. SlotNumber）以及 Name == 'PS' + Instance 字段分离（redfish/plugins/orchestrator/power.lua:336）的命名方式。
修复建议：若平台组件名可能为 PSU1/PSU2 或仅有 PS，建议补充匹配，例如将条件扩展为 string.match(Name, '^PSU?%d+$') or Name == 'PowerSupply' or Name == 'PS'，避免后续平台切换命名时再次聚合失败。
人工检视报告
感谢 @byosoft_zhangchao 的贡献！本次修复针对 SNMP powerSupplyEntireStatus 聚合失败问题，改动精准、范围最小，方向正确。
📋 检视概览
| 项目 | 详情 |
|---|---|
| 评级 | 良好 |
| 严重问题 | 0 |
| 警告 | 1 |
| 建议 | 1 |
已确认修复的问题 ✅
无 AI 机器人历史检视评论。
需关注的问题（已关联代码行）
| # | 文件 | 行 | 级别 | 说明 |
|---|---|---|---|---|
| 1 | interface_config/snmp/plugins/orchestrator/power_supply_info.lua |
26 | ⚠️ 警告 | Redfish 侧 power.lua:383 get_chassis_ps_component_health() 仍只匹配 'PowerSupply'，若同一平台组件名采用 PS1/PS2 格式，Redfish 侧聚合会复现同样问题，建议同步适配 |
| 2 | interface_config/snmp/plugins/orchestrator/power_supply_info.lua |
26 | 💡 建议 | 正则 ^PS%d+$ 未覆盖仓库其他插件用到的 PSU<slot>（upgrade.lua:65）与 Name == 'PS' + Instance（power.lua:336）命名方式，可考虑扩展匹配以防后续平台切换再失效 |
最终结论
本次修复逻辑正确，Lua 正则 ^PS%d+$ 合法且与 PowerSupply 兜底匹配处理得当，CHANGELOG 与 mds/service.json 版本号（1.110.129）同步一致。建议关注上述 Redfish 侧的对称问题。整体可合入。
| 任务名称 | 结果 |
|---|---|
| pre-commit检查 | ❌FAILED |
| 防投毒检查 | ✅SUCCESS |
| 开源合规检查 | ✅SUCCESS |
| 编译构建 | ✅SUCCESS |
| ipmi修改 | ✅SUCCESS |
| 修改行数检查 | ✅SUCCESS |
| AI Review | ✅SUCCESS |
当前PR是否有AI参与:
[x] 否
[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :
PR功能描述 / 为什么需要这个合入:
修改方案:
验证方案:
测试建议:
自验证报告:
该PR关联的issue
(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #
希望检视人员了解: