Skip to content

<chore>[sdk]: Update sdk#3283

Open
ZStack-Robot wants to merge 1 commit into5.5.6from
sync/ye.zou/ZSTAC-81309@@2
Open

<chore>[sdk]: Update sdk#3283
ZStack-Robot wants to merge 1 commit into5.5.6from
sync/ye.zou/ZSTAC-81309@@2

Conversation

@ZStack-Robot
Copy link
Collaborator

Resolves: ZSTAC-81309

Change-Id: I69716d637666776c746570766f687873756d656b

sync from gitlab !9103

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

总体概述

数据库schema迁移,为ModelServiceVO表添加新的version列(VARCHAR(255)类型),记录之前缺少该实体字段的迁移信息。

变更清单

变更类别/文件 摘要
数据库Schema迁移
conf/db/upgrade/V5.5.6__schema.sql
为ModelServiceVO表新增version列(VARCHAR(255)),补充之前缺失的entity字段迁移,不涉及其他功能逻辑改动。

评估代码审查工作量

🎯 1 (简单) | ⏱️ ~3 分钟

庆祝诗

🐰 在云烟中跳跃欢欣
schema版本迎新生
一列添来暖如春
数据库里新光临 🌟


Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)
Check name Status Explanation Resolution
Title check ❌ Error PR标题描述不准确。标题为'Update sdk',但实际改动是向ModelServiceVO表添加版本列的数据库schema迁移,与SDK更新无关。 建议将标题改为更准确地反映实际改动,例如'[database]: Add version column to ModelServiceVO table'或'[schema]: Add version field to ModelServiceVO'。
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed PR描述包含JIRA工单号、Change-Id和GitLab同步信息,虽然信息有限,但与实际的数据库schema迁移改动相关。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sync/ye.zou/ZSTAC-81309@@2

Comment @coderabbitai help to get the list of available commands and usage tips.

Resolves: ZSTAC-81309

Change-Id: I69716d637666776c746570766f687873756d656b
Signed-off-by: AlanJager <ye.zou@zstack.io>
@MatheMatrix MatheMatrix force-pushed the sync/ye.zou/ZSTAC-81309@@2 branch from a491618 to ec1fbaa Compare February 3, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants