SQL上线检测和提交

[peixubin]

在SQL上线检测时出现错误的情况，是否应该不允许提交，至少应该有个开关控制，不允许强制提交。

[LeoQuote]

有相关配置，可以探索下

[peixubin]

具体是哪个配置项？ 我在《配置管理->SQL上线》部分看了，没有合适的

[peixubin]

找到了问题原因。
在AuditV2类中，把自动通过和自动审核等价了。这应该是个严重BUG了，希望能尽快修复。
def generate_audit_setting(self) -> AuditSetting: if self.is_auto_review(): return AuditSetting(auto_pass=True) ... def create_audit(self): ... audit_setting = self.generate_audit_setting() ... # 自动通过的情况 if audit_setting.auto_pass: self.audit.current_status = WorkflowStatus.PASSED

[LeoQuote]

我归纳下, 当前数据库如果设置了"自动审批", 也就是 "AUTO_REVIEW", 那么只要提交的语句不命中设置中的正则, 即使规则引擎判定该语句不能提交, 这个语句也是可以提交成功的.

这确实是实现上的问题, 应该在

Archery/sql/utils/workflow_audit.py

Line 258 in 24f86de

def create_audit(self) -> str:

这个方法里, 尽快判断当前的workflow 状态, 如果是 autoreviewwrong, 则直接 raise exception.

欢迎 pr 哈. 我暂时没空修这个

[peixubin]

参考 #2692，看上去是修复了

[peixubin]

只看到critical_ddl_regex和auto_review_wrong像是实现类似的功能。
critical_ddl_regex是通过正则来匹配语句，和我的需求不一致；
auto_review_wrong看上去像，但是我配置成2,不起作用

[lanheader]

https://hanchuanchuan.github.io/goInception/zh/rules.html#ddl%E6%93%8D%E4%BD%9C 看看goinception的配置

[peixubin]

上线脚本已经通过goinception发现了错误，在SQL提交时会提示“存在N个错误”，让用户确认。用户确认后，存在错误的脚本就会提交到审核（有些还会自动审核通过）。
我的意思是存在错误的脚本，不能提交到审核或自动审核通过


。

[peixubin]

想要达到的效果：

[feiazifeiazi]

强制提交了，有什么关系？ 会自动审核不通过的。

你这里为什么会自动审核通过，我擦，咋通过的？

[peixubin]

我也很奇怪，SQL上线相关配置项：

是开启了自动审核的。
如果我在AUTO_REVIEW_REGEX加入"^insert"，就是把insert语句需要人工审核来处理，那么提交后工作流状态将是“自动审核不通过”，是正确的。但是这样的话，所有的insert都会进入人工审核环节，这不符合我的业务需求，我的目的是DDL或更新行数比较多的要人工审核。