Eli Mesika has submitted this change and it was merged.
Change subject: core: moving AuditLog message length check to DB
......................................................................
core: moving AuditLog message length check to DB
The check in the setMessage breaks API tests since the created mock has
no access to configuration.
Since this check is relevant only for External Events, it was moved to
the stored procedure.
Change-Id: Ib2805ee22301d75a7d6a2c852a3acfe3fc0f5f59
Signed-off-by: Eli Mesika <emesika(a)redhat.com>
---
M backend/manager/dbscripts/audit_log_sp.sql
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/AuditLog.java
2 files changed, 9 insertions(+), 10 deletions(-)
Approvals:
Eli Mesika: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/10522
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2805ee22301d75a7d6a2c852a3acfe3fc0f5f59
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>