-
Notifications
You must be signed in to change notification settings - Fork 213
feat(workflow): refactor workflow execution and notification for auto-created workflows #3201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
PR Reviewer Guide 🔍(Review updated until commit df22a03)
|
PR Code Suggestions ✨No code suggestions found for the PR. |
…-created workflows
a9347aa to
f0c4ff5
Compare
|
Persistent review updated to latest commit f0c4ff5 |
PR Code Suggestions ✨No code suggestions found for the PR. |
|
Persistent review updated to latest commit df22a03 |
PR Code Suggestions ✨No code suggestions found for the PR. |
User description
关联的 issue
https://github.com/actiontech/sqle-ee/issues/2623
描述你的变更
工单webhook通知时,区分通过cloudbeaver工作台非DQL自动创建工单的情况
确认项(pr提交后操作)
Tip
请在指定复审人之前,确认并完成以下事项,完成后✅
not_compatibleneed_update_docDescription
新增针对自动创建工单的执行函数
扩展工单通知类型为 auto_exec_success/auto_exec_failed
修改 ExecuteWorkflow/ExecuteTasksProcess 增加 isAutoCreated 参数
更新注释说明及调用逻辑区分自动和普通工单
Diagram Walkthrough
File Walkthrough
workflow.go
调整工单执行函数支持自动创建逻辑sqle/api/controller/v1/workflow.go
notification.go
扩展工单消息通知支持自动执行类型sqle/notification/notification.go
workflow_schedule.go
扩展任务执行流程区分自动与普通工单sqle/server/workflow_schedule.go