合并受阻
欢迎来到 openUBMC 社区
Hey @iss_renping , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用，可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则，更多详情，请参阅机器人自助配置使用手册。
联系指引
有疑问可以联系 SIG: sig-interface ,
维护者是： @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是： @L_Ling, @PLUS_ONE, @yelvsub .
关联 Issue 通知
@iss_renping , 提交的PR需要关联issue，指导链接：https://forum.openubmc.cn/t/topic/208
如果已经关联了issue, 但是needs-issue标签仍然存在，你可以在评论 /check-issue 来移除它。
The pipeline(pipeline number:2060) is running. >>> Check Pipeline <<<.Please wait a moment...
/ai review
⚠️ 警告：psu_cnt 已变为未使用变量，且槽位列表全部为空时会把已发现的真实 PSU 一并丢弃
改为按 all_slots（合并去重后的实际槽位）遍历后，第 210 行 get_psu_cnt_and_slot(input) 返回的 psu_cnt 在调用方已不再使用（luacheck 也会报 unused local），get_psu_cnt_and_slot 内部仍统计该值，属于遗留死变量。
更需要注意的行为回退：若 psu_slots 与 psu_board_slots 均为空（例如 get_component_info 中 input[3] 槽位字节串为空）而 psu_cnt > 0，则 all_slots 为空、本循环不会执行，psu_map / psu_metrics_map 中已发现的真实 PSU（如 psu_map['PSU1']）将不再输出到 supply_list；而旧代码以 1..psu_cnt 为界遍历，这些真实 PSU 仍会被渲染。这会导致 PSU 信息在槽位枚举数据缺失时静默丢失。
修复建议：
在 all_slots 为空且 psu_cnt > 0 时回退按 1..psu_cnt 兜底遍历（仍复用上面的按槽位命名逻辑），或删除不再使用的 psu_cnt 并以真实槽位遍历为主、增加空列表兜底，确保槽位数据缺失时已有真实 PSU 不被丢弃。
人工检视报告
感谢 @iss_renping 的贡献！本次 PR 支持长城（Great Wall）CRPS 电源适配：将 PSU 遍历逻辑从「按 psu_cnt 顺序号 1..N 遍历」改为「合并 psu_slots 与 psu_board_slots、去重排序后按实际槽位号遍历」，从而在槽位号不连续时也能正确输出对应槽位的 PSU。整体思路合理，CHANGELOG 与 mds/service.json 版本号 1.110.129 保持一致。
📋 检视概览
| 项目 | 详情 |
|---|---|
| 评级 | 需改进 |
| 严重问题 | 0 |
| 警告 | 2 |
| 建议 | 0 |
⚠️ 需关注的问题（已关联代码行）
| # | 文件 | 行 | 级别 | 说明 |
|---|---|---|---|---|
| 1 | interface_config/web_backend/plugins/orchestrator/power.lua |
172 | ⚠️ 警告 | 空行被改成含单个空格的行，约 172/175/180/190/193 行引入行尾空格，违反仓库 trailing-whitespace pre-commit 钩子 |
| 2 | interface_config/web_backend/plugins/orchestrator/power.lua |
229 | ⚠️ 警告 | psu_cnt 变为未使用变量（luacheck 会告警 unused）；当两个槽位列表均为空而 psu_cnt > 0 时，all_slots 为空导致真实 PSU 不再输出（行为回退），建议增加空列表兜底 |
✅ 确认项
- CHANGELOG 新增
[1.110.129]条目、mds/service.json版本号同步升至 1.110.129，二者一致。 - 合并去重+排序的遍历逻辑本身使槽位号不连续（如 1/3/5）时也能正确渲染，符合本次适配目标。
结论
功能改动方向正确，建议合入前修复 2 条警告（清理行尾空格；为 psu_cnt 空槽位场景增加兜底，避免真实 PSU 静默丢失），之后即可合并。
| 任务名称 | 结果 |
|---|---|
| pre-commit检查 | ❌FAILED |
| 防投毒检查 | ✅SUCCESS |
| 开源合规检查 | ✅SUCCESS |
| 编译构建 | ✅SUCCESS |
| ipmi修改 | ✅SUCCESS |
| 修改行数检查 | ✅SUCCESS |
| AI Review | ✅SUCCESS |
当前PR是否有AI参与:
[x] 否
[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :
PR功能描述 / 为什么需要这个合入:
新增支持 长城crps 电源适配
修改方案:
1、合并 psu_slots 和 psu_board_slots，去重排序，用实际槽位号遍历
验证方案:
测试建议:
自验证报告:
该PR关联的issue
(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #
希望检视人员了解: