Skip to content

<fix>[zdfs]: add auth token schema column#4495

Open
zstack-robot-2 wants to merge 1 commit into
5.5.28from
sync/xinhao.huang/fix/ZSTAC-86638-zdfs-auth-schema
Open

<fix>[zdfs]: add auth token schema column#4495
zstack-robot-2 wants to merge 1 commit into
5.5.28from
sync/xinhao.huang/fix/ZSTAC-86638-zdfs-auth-schema

Conversation

@zstack-robot-2

Copy link
Copy Markdown
Collaborator

问题

ZDFS token 修复在 ZdfsVO 上新增 authToken 字段,5.5.28 现有环境的 ZdfsVO 表没有对应列,直接替换 premium jar 会导致数据库字段缺失。

修复

  • conf/db/upgrade/V5.5.28__schema.sql 中增加 ZdfsVO.authToken 升级列。
  • 使用 CALL ADD_COLUMN,兼容已有环境和重复执行场景。

验证

  • schema-only change,未运行编译。

sync from gitlab !10456

Add the ZdfsVO.authToken upgrade column required by the ping config token flow.

Jira: http://jira.zstack.io/browse/ZSTAC-86638

Test: not run, schema-only change

Change-Id: I636378e244b458d8e5063ebc496820739a546087
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Could not fetch remote config from http://open.zstack.ai:20001/code-reviews/zstack-cloud.yaml: TimeoutError: The operation was aborted due to timeout
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 6e933e28-6a55-407c-b946-e2d7a66bd774

📥 Commits

Reviewing files that changed from the base of the PR and between 1fde321 and c26102f.

📒 Files selected for processing (1)
  • conf/db/upgrade/V5.5.28__schema.sql

Walkthrough

该变更在数据库升级脚本 V5.5.28__schema.sql 中新增了一条 ADD_COLUMN 调用,为 ZdfsVO 表添加了 authToken 列,类型为 varchar(255),默认值为 NULL。

Changes

数据库架构变更

Layer / File(s) Summary
新增authToken列迁移
conf/db/upgrade/V5.5.28__schema.sql
通过 ADD_COLUMN 调用为 ZdfsVO 表新增 authToken 列(varchar(255)),默认值设为 NULL。

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

一列新增小小事,
authToken 悄悄落地,
数据库里加把锁,
兔子蹦跳来贺喜 🐇
升级脚本又添一笔!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 标题准确概括了为 ZDFS 新增 auth token 架构列这一主要变更。
Description check ✅ Passed 描述与变更一致,说明了新增 ZdfsVO.authToken 列、兼容重复执行及仅为 schema 变更。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sync/xinhao.huang/fix/ZSTAC-86638-zdfs-auth-schema

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

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.

1 participant