Eli Mesika has posted comments on this change. Change subject: core: adding unlock_entity.sh script(#868638) ......................................................................
Patch Set 11: (1 inline comment) .................................................... File backend/manager/dbscripts/dbfunctions.sh Line 383: #adds a record to audit_log in case of calling unlock_entity Line 384: log_unlock_entity() { Line 385: local object_type=${1} Line 386: local id=${2} Line 387: msg="User ${USERNAME} run unlock_entity script on ${object_type} ${id}" There may be more than one db user , actually in production it is called engine IIRC anyway, in patch set 12 I am displaying both system & db user Example : System user emesika run unlock_entity script on disk 52c30705-6901-4a2d-a724-bdcc3458f1b0 with db user postgres Line 388: CMD="insert into audit_log(log_time,log_type_name,log_type,severity,message) Line 389: values(now(), 'USER_RUN_UNLOCK_ENTITY_SCRIPT', 2024, 10, '${msg}')" Line 390: execute_command "${CMD}" "${DATABASE}" "${SERVERNAME}" "${PORT}" Line 391: } -- To view, visit http://gerrit.ovirt.org/8688 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f1712fb945933d92ee9a6b97bc71d4ba3a01597 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Itamar Heim <ih...@redhat.com> Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches