已开启
GroupID #1433
jupitergone创建于 22 天前
GroupID #1433
已开启
jupitergone创建于 22 天前
jupitergone
22 天前

当前PR是否有AI参与:

[x] 否

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

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

修改方案**:

验证方案**:

测试建议**:

自验证报告**:

该PR关联的issue

(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #

希望检视人员了解:

likedislike
合并受阻
openUBMC-robotopenUBMC-robot成员
22 天前 将WeliveWelove,DataSculptor,huanghan,aaronlee设为审查人
openUBMC-robotopenUBMC-robot成员
22 天前 添加了label:sig/sig-interface
openUBMC-robot
openUBMC-robot成员
22 天前 评论:

欢迎来到 openUBMC 社区

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

机器人使用手册

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

联系指引

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

likedislike
openUBMC-robotopenUBMC-robot成员
22 天前 添加了label:openUBMC-cla/no
openUBMC-robot
openUBMC-robot成员
22 天前 评论:

CLA 签署引导

@jupitergone , 感谢您的代码贡献。

以下 Commit(s) 没有签署 Contributor License Agreement (CLA)

Commit 原因
0b3c1f03 init 这条 Commit 的邮箱未签署 CLA !请检查下邮箱是否正确配置。

如果您需要签署 CLA,您可以点击此链接去签署 CLA

如果需要检查邮箱是否配置正确, 可以点击 FAQs 去查看使用手册

在完成 CLA 签署或者更新完邮箱后, 需再次评论 /check-cla 来检查 CLA 的签署状态。

likedislike
openUBMC-robotopenUBMC-robot成员
22 天前 添加了label:needs-issue
openUBMC-robot
openUBMC-robot成员
22 天前 评论:

关联 Issue 通知

@jupitergone , 提交的PR需要关联issue,指导链接:https://forum.openubmc.cn/t/topic/208
如果已经关联了issue, 但是needs-issue标签仍然存在,你可以在评论 /check-issue 来移除它。

likedislike
openUBMC-robot
openUBMC-robot成员
22 天前 评论:

/ai review

likedislike
atomgit-bot
atomgit-bot成员
22 天前 评论:

代码审查

Closing Summary

已逐项审查 1 个变更文件:

  • interface_config/redfish/mapping_config/Chassis/Chassis.json — 发现 1 个问题(P2,新增 OEM 字段未同步更新 CSDL/JSON Schema 元数据)。

按优先级统计:

  • P0:0
  • P1:0
  • P2:1
  • P3:0

总体风险判断:该变更在 Chassis.json 内部自洽(字段名、Range 校验、D-Bus 方法名与属性映射均一致,未发现索引错位或拼写错误),但存在 OEM 元数据/JSON Schema 未同步的契约不一致,可能导致依赖 $metadata / Schema 校验的 Redfish 客户端或一致性工具拒绝新增字段,建议合入前补齐对应 schema 定义,整体风险为中等。

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

💬 仅评论

likedislike
openUBMC-robot
openUBMC-robot成员
22 天前 评论:

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

likedislike
openUBMC-robotopenUBMC-robot成员
22 天前 添加了label:ci-running
openUBMC-robotopenUBMC-robot成员
22 天前 删除了label:ci-running
openUBMC-robot
openUBMC-robot成员
22 天前 评论:
任务名称结果
pre-commit检查✅SUCCESS
防投毒检查✅SUCCESS
开源合规检查✅SUCCESS
编译构建✅SUCCESS
ipmi修改✅SUCCESS
修改行数检查✅SUCCESS
likedislike
openUBMC-robotopenUBMC-robot成员
22 天前 添加了label:ci-successful
atomgit-bot
atomgit-bot成员22 天前进行代码检视1
interface_config/redfish/mapping_config/Chassis/Chassis.json
@@ -551,2 +599,4 @@
551599 "ServerIndex": "${ProcessingFlow[20]/Destination/ServerIndex}",
552600 "SuperPodSize": "${ProcessingFlow[20]/Destination/SuperPodSize}",
601+ "NPUScaleUpGroupId": "${ProcessingFlow[20]/Destination/NPUScaleUpGroupId}",
602+ "NPUScaleOutGroupId": "${ProcessingFlow[20]/Destination/NPUScaleOutGroupId}",
atomgit-bot
atomgit-bot22 天前评论:

🟡 Medium Priority

本 diff 在 Chassis.json 中给 SuperPodLabel 对象新增了 NPUScaleUpGroupIdNPUScaleOutGroupId 两个 OEM 属性(PATCH 请求体、GET 响应体、D-Bus 属性映射三处同时加入),但对应的 OEM 元数据/模式文件未同步更新:

建议:同步修改 CSDL 元数据与 JSON Schema:在 openubmcchassis_v1.xml 的 SuperPodLabel ComplexType 和 openubmcchassis.v1_0_0.json 的 definitions/SuperPodLabel 中补充 NPUScaleUpGroupId、NPUScaleOutGroupId 两个整数属性(范围 0–65535、ReadWrite/readonly=false),与映射配置保持一致,避免 additionalProperties=false 下响应字段被判定为非法。

likedislike