已合并
【b3d6ad33c88b64eae08b4ba99f0f235d】feat(cli): 社区软件包剪裁psu work mode的cli接口 #1426
【b3d6ad33c88b64eae08b4ba99f0f235d】feat(cli): 社区软件包剪裁psu work mode的cli接口 #1426
已合并
tangjingman创建于 8月11日
tangjingman
tangjingman成员
8月11日

当前PR是否有AI参与:

[x] 否

[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :

PR功能描述 / 为什么需要这个合入**:

裁剪电源模式相关的cli接口

修改方案**:

(1)社区软件包发送ipmcget -d psuinfo,返回信息不包含电源模式和深度休眠信息
(2)社区软件包发送ipmcset -d psuworkmode,返回提示的参数信息不包含psuworkmode这个参数;
(3)平台软件包发送ipmcget -d psuinfo,返回信息包含电源模式和深度休眠信息
(4)平台软件包发送ipmcset -d psuworkmode,返回提示的参数信息包含psuworkmode支持的所有参数;

验证方案**:

(1)社区软件包发送ipmcget -d psuinfo,返回信息不包含电源模式和深度休眠信息;
(2)社区软件包发送ipmcset -d psuworkmode,返回提示的参数信息不包含psuworkmode这个参数;
(3)平台软件包发送ipmcget -d psuinfo,返回信息包含电源模式和深度休眠信息
(4)平台软件包发送ipmcset -d psuworkmode,返回提示的参数信息包含psuworkmode支持的所有参数;

测试建议**:

自验证报告**:

该PR关联的issue

resolves#957

希望检视人员了解:

likedislike
Pull Request已成功合入, 合并人@openUBMC-robot
(感谢 tangjingman 的贡献)
tangjingmantangjingman成员
8月11日 关联了issue:[需求]: 支持cli接口相关散热管理/功耗管理特性裁剪
openUBMC-robotopenUBMC-robot成员
8月11日 将WeliveWelove,DataSculptor,huanghan,aaronlee设为审查人
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:sig/sig-interface
openUBMC-robot
openUBMC-robot成员
8月11日 评论:

欢迎来到 openUBMC 社区

Hey @tangjingman , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 说明文档论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。

联系指引

有疑问可以联系 SIG: sig-interface ,
维护者是: @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是: @L_Ling, @PLUS_ONE, @yelvsub .

likedislike
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:openUBMC-cla/yes
openUBMC-robot
openUBMC-robot成员
8月11日 评论:

The pipeline(pipeline number:1717) is running. >>> Check Pipeline <<<.Please wait a moment...

likedislike
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:ci-running
openUBMC-robot
openUBMC-robot成员
8月11日 评论:

/ai review

likedislike
atomgit-bot
atomgit-bot成员
8月11日 评论:

代码审查

审查总结

逐文件审查结果:

文件 审查结论
interface_config/cli/echoes/ipmcget/_psuinfo 无问题——条件性隐藏 PSU 工作模式头部和预测状态头部的逻辑正确,cjson.null 判断与社区配置的 null 值一致
interface_config/cli/ipmcget/target.json 无问题——电源相关字段置 null、移除 Property ProcessingFlow 步骤、GetPowerList.Input 索引从 [2] 更新为 [1] 均正确(社区配置独立使用时)
interface_config/cli/ipmcset/target.json 无问题——psuworkmode 端点完整删除,干净利落
mds/service.json 无问题——常规版本号从 1.110.99 升级到 1.110.100
oem/huawei/cli/config.json 发现 1 个问题(P2)——ProcessingFlow 索引冲突
oem/huawei/cli/ipmcset/psuworkmode.json 无问题——结构与原始配置一致(增加 Resources 包装层,与 target.json 顶层格式匹配),内部 ProcessingFlow 索引自洽

问题统计:P0: 0 / P1: 0 / P2: 1 / P3: 0

总体风险评估:中低风险。社区侧裁剪逻辑自洽,OEM 侧存在一处索引协调缺陷——OEM 配置向 ProcessingFlow 插入 Property 步骤时未同步更新社区配置中保留的 GetPowerList.Input 引用,可能导致 OEM 平台包中 psuinfo 的电源列表或电源模式字段异常。该问题取决于 config 引擎的 ProcessingFlow 合并语义,建议与引擎开发者确认后修正。

类型 数量
🔴 阻塞 0
🟡 建议 1

💬 仅评论

likedislike
不准确?
tangjingman
tangjingman成员
8月11日 评论:

/publish

likedislike
openUBMC-robotopenUBMC-robot成员
8月11日 删除了label:ci-running
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:ci-successful
tangjingmantangjingman成员
8月11日 修改标题为 “feat(cli): 社区软件包剪裁psu work mode的cli接口”,原标题为“feat(snmp): 社区软件包剪裁psu work mode的cli接口”
tangjingmantangjingman成员
8月11日 强制推送  1 个提交:d3ecaf1a-feat(cli): 社区软件包剪裁psu work mode的cli接口
openUBMC-robotopenUBMC-robot成员
8月11日 删除了label:ci-successful
openUBMC-robot
openUBMC-robot成员
8月11日 评论:

The pipeline(pipeline number:1718) is running. >>> Check Pipeline <<<.Please wait a moment...

likedislike
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:ci-running
openUBMC-robotopenUBMC-robot成员
8月11日 删除了label:ci-running
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:ci-successful
atomgit-bot
atomgit-bot成员8月11日进行代码检视2
oem/huawei/cli/config.json
@@ -0,0 +45,4 @@
45+ }
46+ }
47+ ]
48+ }
atomgit-bot
atomgit-bot8月11日评论:

🟡 Medium Priority

变更后的社区 target.json 中,GetPowerList.Input 引用 ${ProcessingFlow[1]/Destination/PowerList}(ProcessingFlow 仅含 List 步骤,1-based 索引 [1] 正确指向 List)。

OEM 的 config.json 通过 Modify 向 ProcessingFlow 添加 Property 步骤,但其 RspBody 中 6 个字段均引用 ${ProcessingFlow[1]/Destination/...},同时未更新社区配置中保留的 GetPowerList.Input

根据合并语义不同,将产生以下任一失败:

  • 若 Property 被 prepend 到索引 [1]GetPowerList.InputProcessingFlow[1])错误指向 Property 步骤,Property 的 Destination 不含 PowerList,导致 GetPowerList 语句失败。
  • 若 Property 被 append 到索引 [2]:OEM 的 RspBody 字段(ActiveStandbySupported 等)引用 ProcessingFlow[1] 错误指向 List 步骤,List 的 Destination 不含这些字段,导致电源模式信息缺失。
  • 若 Modify 的 ProcessingFlow 为整体替换:List 步骤丢失,GetPowerList 同样失败。

无论哪种合并策略,OEM 配置均无法同时保证社区保留的 GetPowerList.Input 与 OEM 新增的 RspBody 字段引用正确。

likedislike
不准确?
tangjingman
tangjingman成员
8月12日 评论:

合并策略是:先进行的数据计算,再进行的属性值修改,实际出包测试不存在上述问题

tangjingmantangjingman成员
8月11日 修改了pull request 的描述
openUBMC-robotopenUBMC-robot成员
8月11日 添加了label:unresolved-reviews
openUBMC-robot
openUBMC-robot成员
8月11日 评论:

注意

当前PR有 1 条评论意见. 其中 1 条未处理完毕. 全部处理完后, 请评论 /check-pr 进行检查. 😄

likedislike
tangjingmantangjingman成员
8月12日 解决了最后一个问题
tangjingman
tangjingman成员
8月12日 评论:

/check-pr

likedislike
openUBMC-robotopenUBMC-robot成员
8月12日 删除了label:unresolved-reviews
openUBMC-robot
openUBMC-robot成员
8月12日 评论:

合入校验失败

@tangjingman, 当前PR不可合入,原因如下:

标签没满足合入条件
  • PR合入需要以下标签 1 lgtm, 但是当前只有0.
  • PR合入需要以下标签 1 approved, 但是当前只有0.
标签使用技巧
  • lgtm: 合并PR时必要的标签。代码仓管理人员可以通过评论 /lgtm 来添加标签。PR创建者可以通过评论 /lgtm cancel 来删除标签,但无法通过评论 /lgtm 添加标签。
  • approved: 合并PR时必要的标签。代码仓管理人员可以通过评论 /approve 来添加标签,通过评论 /approve cancel 来删除标签。
likedislike
tangjingmantangjingman成员
8月13日 强制推送  7 个提交:f90eb876-6 commits from branch main689d17a7-feat(cli): 社区软件包剪裁psu work mode的cli接口
openUBMC-robotopenUBMC-robot成员
8月13日 删除了label:openUBMC-cla/yes
openUBMC-robotopenUBMC-robot成员
8月13日 添加了label:openUBMC-cla/yes
openUBMC-robot
openUBMC-robot成员
8月13日 评论:

CLA 签署成功

tangjingman, 感谢您的代码贡献。 所有的代码贡献者都已完成了 CLA 签署。 👍

likedislike
openUBMC-robotopenUBMC-robot成员
8月13日 删除了label:ci-successful
openUBMC-robot
openUBMC-robot成员
8月13日 评论:

The pipeline(pipeline number:1732) is running. >>> Check Pipeline <<<.Please wait a moment...

likedislike
openUBMC-robotopenUBMC-robot成员
8月13日 添加了label:ci-running
openUBMC-robotopenUBMC-robot成员
8月13日 删除了label:ci-running
openUBMC-robot
openUBMC-robot成员
8月13日 评论:
任务名称结果
pre-commit检查✅SUCCESS
防投毒检查✅SUCCESS
开源合规检查✅SUCCESS
编译构建✅SUCCESS
ipmi修改✅SUCCESS
修改行数检查✅SUCCESS
likedislike
openUBMC-robotopenUBMC-robot成员
8月13日 添加了label:ci-successful
wanghang
wanghang成员
8月13日 评论:

/lgtm

likedislike
openUBMC-robot
openUBMC-robot成员
8月13日 评论:

标签指令手册

  • 谢谢您的检视, 您的意见对于我们很重要. 仓库 maintainers 会认真考虑您的意见.
  • 您当前没有添加标签 lgtm 的权限.
likedislike
zhangfuhai
zhangfuhai成员
8月13日 评论:

/lgtm

likedislike
openUBMC-robot
openUBMC-robot成员
8月13日 评论:

标签指令手册

  • 谢谢您的检视, 您的意见对于我们很重要. 仓库 maintainers 会认真考虑您的意见.
  • 您当前没有添加标签 lgtm 的权限.
likedislike
lixiaoning成员
8月13日 评论:

/lgtm
/approve
/publish
/check-pr

likedislike
openUBMC-robot
openUBMC-robot成员
8月13日 评论:

合入校验失败

@L_Ling, 当前PR不可合入,原因如下:

标签没满足合入条件
  • PR合入需要以下标签 1 lgtm, 但是当前只有0.
  • PR合入需要以下标签 1 approved, 但是当前只有0.
标签使用技巧
  • lgtm: 合并PR时必要的标签。代码仓管理人员可以通过评论 /lgtm 来添加标签。PR创建者可以通过评论 /lgtm cancel 来删除标签,但无法通过评论 /lgtm 添加标签。
  • approved: 合并PR时必要的标签。代码仓管理人员可以通过评论 /approve 来添加标签,通过评论 /approve cancel 来删除标签。
likedislike
openUBMC-robotopenUBMC-robot成员
8月13日 添加了label:approvedlgtm
openUBMC-robot
openUBMC-robot成员
8月13日 评论:

  • 标签 lgtm, approved 已被添加. 这意味着 L_Ling 已经检查过代码了. 👋
Tips
  • 如果PR还没有被合入, 请评论 /check-pr 进行检查. 😄
likedislike
openUBMC-robotopenUBMC-robot成员
8月13日 合入了pull request,合并节点 SHA:616f0f40e17b18a6f9eb01b96c0f18cb25d82897
openUBMC-robotopenUBMC-robot成员
8月13日 修改标题为 “【b3d6ad33c88b64eae08b4ba99f0f235d】feat(cli): 社区软件包剪裁psu work mode的cli接口”,原标题为“feat(cli): 社区软件包剪裁psu work mode的cli接口”
openubmc-ci-bot成员
8月13日 评论:

This component has been successfully published.

likedislike