Skip to content

Conversation

@MatheMatrix
Copy link
Owner

DBImpact

Resolves: ZSTAC-11

Change-Id: I6c6f66796372726772746d74616c6f63776e7270

sync from gitlab !9038

DBImpact

Resolves: ZSTAC-11

Change-Id: I6c6f66796372726772746d74616c6f63776e7270
Resolves: ZSTAC-11

Change-Id: I6868677567686d636b6d63626e6161797a727064
@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

总体概述

引入资源通知订阅相关的数据库表定义(ResNotifySubscriptionVO 和 ResNotifyWebhookRefVO),并在测试库的 ApiHelper 中添加大量新的 API 方法来支持订阅、查询和删除操作。

变更内容

类别 / 文件 变更摘要
数据库模式定义
conf/db/upgrade/V5.6.0__schema_resource_lifecycle.sql
新增两张表:ResNotifySubscriptionVO(存储订阅元数据,包含 uuid、名称、描述、资源/事件类型、订阅状态)和 ResNotifyWebhookRefVO(存储 Webhook 引用信息,包含 webhookUrl、secret、自定义头、投递时间、失败计数),具有级联删除的外键关系
API 辅助方法扩展
testlib/src/main/java/org/zstack/testlib/ApiHelper.groovy
添加 110+ 行新的公共方法,包括 deleteResNotifySubscriptionqueryResNotifySubscriptionsubscribeResNotify 等,均遵循统一的模式:初始化操作对象、设置会话、配置策略、调用闭包、处理响应

评估代码审查工作量

🎯 2 (Simple) | ⏱️ ~12 分钟

兔兔的庆贺诗

🐰 新表降临在数据库地,
Webhook 订阅显神采,
ApiHelper 方法百般齐,
级联删除保数据完美,
通知系统今日启新篇!✨


Important

Pre-merge checks failed

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

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive 标题 '[conf]: 11' 不清楚且过于简洁,仅用数字'11'作为描述,没有传达变更的具体内容,无法理解此PR的实际目的。 请使用更具体的描述替换'11',例如 '[conf]: Add resource notification subscription schema' 来清楚说明此PR的主要变更。
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed PR描述包含'DBImpact'、'Resolves: ZSTAC-11'和'sync from gitlab !9038'等信息,与变更内容相关,描述了数据库影响和相关的追踪信息。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

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.

3 participants