Open
[需求]: WEB后端支持SP升级带内固件 #5
lisiguangAdd on 14 days ago
14 days ago Added label: sig/sig-interface
openUBMC-robot
14 days ago Comments：
14 days ago Comments：
欢迎来到 openUBMC 社区
Hey @aaronlee , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用，可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则，更多详情，请参阅机器人自助配置使用手册。
联系指引
有疑问可以联系 SIG: sig-interface ,
维护者是： @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是： @PLUS_ONE .
openUBMC-robot
14 days ago Comments：
14 days ago Comments：
Issue 工作流介绍
社区的需求和缺陷的工作流状态处于纳管中。
| 类型 | 上个状态 | 下个状态 | 评论命令 | 谁可以操作 |
|---|---|---|---|---|
| 需求 | 待办的 | 已接纳 | /todo2accepted | SIG组: sig-release-management (maintainer) |
| 需求 | 已接纳 | 开发中 | /accepted2developing | 仓库成员角色: Maintainer, Committer, Contributor, Owner |
| 需求 | 开发中 | 待验收 | /developing2UAT | 仓库成员角色: Maintainer, Committer, Contributor, Owner |
| 需求 | 待验收 | 已完成 | /UAT2done | SIG组: sig-QA (maintainer,committer) |
| 需求 | 待办的 | 已挂起 | /todo2suspended | SIG组: sig-release-management (maintainer) |
| 需求 | 已挂起 | 已接纳 | /suspended2accepted | SIG组: sig-release-management (maintainer) |
| 需求 | 待办的 | 已拒绝 | /todo2rejected | SIG组: sig-release-management (maintainer) |
| 需求 | 已拒绝 | 已接纳 | /rejected2accepted | SIG组: sig-release-management (maintainer) |
14 days ago changed custom state from 待办的 to 已接纳
7 event messages are collapsed here。 See more
5 days ago Associated pull request
zhaotianli
4 days ago Comments：
4 days ago Comments：
PR #756 回顾分析: 【09200b1c564ae7c56d7f5e955b3fb058】feat : 资源协作接口 bmc.kepler.Managers.Ums.SP 新增 UpdateFirmwareFromLocal 方法
仓库: openUBMC/mdb_interface
分支: main → main（m0_73888240/mdb_interface → openUBMC/mdb_interface）
合并时间: 2026-11-02 12:35:40 (+08:00)
1. AI 检视意见追踪
本 PR 无 AI 检视记录（PR 描述中勾选"AI 未参与"，AI 检视评论列表为空），无 AI 意见可追踪。
| 指标 | 结果 |
|---|---|
| AI 意见总数 | 0 |
| 已采纳并修复 | 0 |
| 已回复说明 | 0 |
| 未响应 | 0 |
2. 人工检视补充
- 检视人: nobug_nolife（Jiancong_Huo）
- 意见概要:
- SP.json 新增方法块（第 123 行）："mdb_interface我记得不用定义privilege字段的"——指出新增 UpdateFirmwareFromLocal 时不应定义 privilege 字段，mdb_interface 仓无此惯例
- 解决状态: ✅ 已解决
- 开发者 m0_73888240 于 2026-10-29 15:57 推送 diff 第 6 版（commit c94795b2）修改该行，移除 privilege 字段；讨论于 2026-10-29 16:04 由开发者标记 resolved
- 最终合入代码中 UpdateFirmwareFromLocal 无 privilege 字段，与检视意见一致
- 另有 aaronlee 于 2026-11-02 两次执行
/lgtm/approve完成合入门禁
3. 评审贴覆盖情况
评审贴链接：【已评审】资源协作接口新增UpdateFirmwareFromLocal
| 评审点 | 是否涵盖 |
|---|---|
| 评审点1：变更资源协作接口 bmc.kepler.Managers.Ums.SP——新增 UpdateFirmwareFromLocal 方法（请求签名 ssss，入参 ImageFileName/SignFileName/Parameter/ActiveMethod 均为 String，无响应体，新增方法无影响） | ✅ 已涵盖 |
覆盖统计: 1/1（SP.json 新增 22 行完整实现该方法，4 个入参名称、类型与描述均与评审贴一致；另含 CHANGELOG 记录与 mds/service.json 版本号 1.110.66 → 1.110.67 提升。评审结论：已线下审核通过，同意新增方法）
注：评审贴中"访问权限 BasicSetting"在 mdb 层最终以不定义 privilege 字段的方式体现（经人工检视意见澄清仓库惯例），不影响方法签名与功能。
4. 总结
- AI 检视：本 PR 未引入 AI 检视，无 AI 意见追踪数据。
- 开发者响应态度良好：人工检视意见提出后 3 天内（2026-10-26 → 10-29）推送修复版本移除 privilege 字段并标记讨论 resolved，处理闭环完整。
- 人工检视补充价值：nobug_nolife 纠正了新增方法中 privilege 字段的使用，使实现符合 mdb_interface 仓的接口定义惯例。
- 流程观察：PR 合入前因 CLA 标签来源问题多次触发合入校验失败，经
/check-cla重签后成功合入；合入后组件已发布（12:45）。建议后续 PR 提前确认 CLA 签署状态以减少合入阻塞。
此回顾分析由 AI (OpenClaw + GLM-5-turbo) 完成
4 days ago Associated pull request
6 event messages are collapsed here。 See more
21 hours ago Associated pull request
所属版本
openUBMC 26.12
版本分支
openUBMC main
简要说明
支持WEB界面SP工具升级带内固件功能，后端提供升级功能、展示和删除现有升级文件信息功能
前置条件
BMC运行正常
系统用户
运维人员
主成功场景
1、WEB支持通过SP升级带内固件，包括：
1）支持上传固件升级包（.zip格式）和签名文件（.asc、.p7s格式）；
2）支持弹框提示：是否立即到SP管理界面设置下次OS从SP启动？
3）支持固件验签、升级生效；
2、WEB支持展示和删除现有升级文件信息
3、刷新用户指南、在线帮助、接口文档
非功能需求类型
兼容性影响评估
不涉及
扩展场景（含异常场景）
不涉及
周边依赖
不涉及
约束
不涉及