Star41
258
代码
代码
Issues110
Pull Requests39
项目讨论
Wiki
分析
Star41
258
已合并
【4f91f007fc3dd5d19ea87ed549e0081d】fix: 修复web界面长时间显示网卡资源正在卸载 #1425
Szgsh创建于 29 天前
【4f91f007fc3dd5d19ea87ed549e0081d】fix: 修复web界面长时间显示网卡资源正在卸载 #1425
已合并
Szgsh创建于 29 天前
从Szgsh/rackmountmain合入到openUBMC/rackmountmain
S
Szgsh
29 天前

当前PR是否有AI参与:

[x] 否

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

该PR关联的issue

https://gitcode.com/openUBMC/rackmount/issues/1000

likedislike
Pull Request已成功合入, 合并人@openUBMC-robot
（感谢 Szgsh 的贡献）
SSzgsh
29 天前 关联了issue：[缺陷]: fix: 修复web界面长时间显示网卡资源正在卸载
openUBMC-robotopenUBMC-robot成员
29 天前 将WeliveWelove,DataSculptor,huanghan,aaronlee设为审查人
openUBMC-robotopenUBMC-robot成员
29 天前 添加了label：sig/sig-interface
openUBMC-robot
openUBMC-robot成员
29 天前 评论：

欢迎来到 openUBMC 社区

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

机器人使用手册

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

联系指引

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

likedislike
openUBMC-robotopenUBMC-robot成员
29 天前 添加了label：openUBMC-cla/yes
S
Szgsh
29 天前 评论：

/publish

likedislike
openUBMC-robot
openUBMC-robot成员
29 天前 评论：

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

likedislike
openUBMC-robotopenUBMC-robot成员
29 天前 添加了label：ci-running
openUBMC-robot
openUBMC-robot成员
29 天前 评论：

/ai review

likedislike
atomgit-bot
atomgit-bot成员
29 天前 评论：

代码审查

✅ 未发现问题

likedislike
不准确？
openUBMC-robotopenUBMC-robot成员
29 天前 删除了label：ci-running
openUBMC-robotopenUBMC-robot成员
29 天前 添加了label：ci-successful
H
huangleisheng成员29 天前进行代码检视2
interface_config/web_backend/script/Systems/validate_preload_port_count.lua
已过期
@@ -9,6 +9,10 @@
99-- See the Mulan PSL v2 for more details.
1010 
1111-- 校验当前网卡是否具备PortConfig接口
12+if not ReqBody or not ReqBody.PreloadPortCount or type(ReqBody.PreloadPortCount) == 'userdata' then
H
huangleisheng29 天前评论：

这个判断的意图是：当请求体未携带 PreloadPortCount（例如仅携带 AdapterId 触发 StartRemovingDevice 卸载流程）时，跳过该校验，避免因 PortConfig 路径缺失而报 PropertyModificationNotSupported，从而解决 web 界面长时间显示“正在卸载”。用 type(...) == 'userdata' 已能覆盖框架对未携带字段的 userdata 哨兵表示，配合前面 not ReqBody.PreloadPortCount（nil/false）在功能上是正确的。

建议评估是否改用 type(...) ~= 'number'，可一并兜底字符串等非数值非法入参直接跳过校验；但因 ReqBody 中 PreloadPortCount 的 Type 已声明为 integer，框架通常会在更早阶段拦截非数值，二者实际差异很小。此为可选优化，非阻塞项，请确认是否调整或说明维持现状的原因。

🤖 由 gitcode-rackmount-pr-review + deepseek-v4-flash 检视

likedislike
System
系统消息系统
27 天前 评论：

changed this line on 217095d5 view diff detail

H
huangleisheng成员
29 天前 评论：

PR #1425 Code Review: 修复web界面长时间显示网卡资源正在卸载

仓库: openUBMC/rackmount
分支: main → main
关联: issue #1000（[缺陷] 修复web界面长时间显示网卡资源正在卸载）
关联设计: 无设计 topic 链接，跳过设计一致性核对

变更概览

文件 变更 说明
CHANGELOG.md +3 新增 [1.110.100] 2026-08-10 记录
mds/service.json +1/-1 版本号 1.110.99 → 1.110.100
validate_preload_port_count.lua +4 请求未携带 PreloadPortCount 时跳过校验

逐项 Review

  • 版本一致性：CHANGELOG 与 mds/service.json 均更新为 1.110.100，日期 2026-08-10 与 PR 创建时间一致，版本号与上一版本 1.110.99 连续。✓
  • CHANGELOG 格式：## [版本] 日期 + 描述 + 空行的格式与既有条目一致，无格式问题。✓
  • Lua 逻辑正确性：在 not ReqBody or not ReqBody.PreloadPortCount or type(ReqBody.PreloadPortCount) == 'userdata' 时提前 return，跳过后续对 ProcessingFlow[3]（PortConfig 路径）的校验。当请求仅携带 AdapterId 触发 StartRemovingDevice 卸载流程、未携带 PreloadPortCount 时，不再因 PortConfig 路径缺失而报 PropertyModificationNotSupported，从而修复 web 界面长时间显示“正在卸载”。逻辑正确。✓
  • 边界：PreloadPortCount 为合法数字（含 0）时，Lua 中 0 为 truthy，not ReqBody.PreloadPortCount 为 false，不会误跳过校验，正常进入后续 AllowableValues 校验。✓
  • ProcessingFlow 索引：脚本使用 ProcessingFlow[3]（PortConfig Property 步骤）与 ProcessingFlow[4]（SetPortCount 步骤），与 NetworkAdapter.json PATCH 的 ProcessingFlow 数组索引一致（0-based）。✓
  • 未发现敏感信息、错误链接或格式问题。✓

发现的问题

  1. （既有讨论串，已追问，待作者确认）validate_preload_port_count.lua:12：判断 type(ReqBody.PreloadPortCount) == 'userdata' 是否考虑改为 ~= 'number'，以兜底字符串等非数值非法入参。非阻塞、可选优化，功能上现有实现已正确。

总结

修复逻辑正确、版本一致、变更范围合理（仅 3 个文件，改动小）。1 条既有建议（非阻塞、可选优化）已线程追问，待作者确认后即可合入。

🤖 由 gitcode-rackmount-pr-review + deepseek-v4-flash 检视

likedislike
C
changdexing成员27 天前进行代码检视2
mds/service.json
已过期
@@ -2,3 +2,3 @@
22 "name": "rackmount",
33 "type": "configuration",
4- "version": "1.110.99",
4+ "version": "1.110.100",
C
changdexing27 天前评论：

image.png
现在不要改

likedislike
System
系统消息系统
27 天前 评论：

changed this line on 217095d5 view diff detail

SSzgsh
27 天前 强制推送  1 个提交:217095d5-fix: 修复web界面长时间显示网卡资源正在卸载
openUBMC-robot
openUBMC-robot成员
27 天前 评论：

CLA 签署成功

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

likedislike
openUBMC-robot
openUBMC-robot成员
27 天前 评论：

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

likedislike
openUBMC-robotopenUBMC-robot成员
27 天前 删除了label：ci-successful
openUBMC-robotopenUBMC-robot成员
27 天前 添加了label：ci-running
openUBMC-robotopenUBMC-robot成员
27 天前 添加了label：unresolved-reviews
openUBMC-robot
openUBMC-robot成员
27 天前 评论：

注意

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

likedislike
openUBMC-robot
openUBMC-robot成员
27 天前 评论：

注意

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

likedislike
openUBMC-robotopenUBMC-robot成员
27 天前 删除了label：ci-running
openUBMC-robot
openUBMC-robot成员
27 天前 评论：
任务名称结果
pre-commit检查✅SUCCESS
防投毒检查✅SUCCESS
开源合规检查✅SUCCESS
编译构建✅SUCCESS
ipmi修改✅SUCCESS
修改行数检查✅SUCCESS
likedislike
openUBMC-robotopenUBMC-robot成员
27 天前 添加了label：ci-successful
Hhuangleisheng成员
27 天前 解决了最后一个问题
H
huangleisheng成员
27 天前 评论：

/lgtm
/approve

likedislike
openUBMC-robotopenUBMC-robot成员
27 天前 添加了label：approvedlgtm
openUBMC-robot
openUBMC-robot成员
27 天前 评论：

  • 标签 lgtm, approved 已被添加. 这意味着 yelvsub 已经检查过代码了. 👋
Tips
  • 如果PR还没有被合入, 请评论 /check-pr 进行检查. 😄
likedislike
H
huangleisheng成员
27 天前 评论：

/check-pr

likedislike
openUBMC-robotopenUBMC-robot成员
27 天前 删除了label：unresolved-reviews
openUBMC-robotopenUBMC-robot成员
27 天前 合入了pull request，合并节点 SHA：6161a5c8c9b445c1ce8372815f4878c05ce99bdf
openUBMC-robotopenUBMC-robot成员
27 天前 修改标题为 “【4f91f007fc3dd5d19ea87ed549e0081d】fix: 修复web界面长时间显示网卡资源正在卸载”，原标题为“fix: 修复web界面长时间显示网卡资源正在卸载”
O
openubmc-ci-bot成员
27 天前 评论：

This component has been successfully published.

likedislike