From Moti Asayag <masayag@redhat.com>:
Moti Asayag has submitted this change and it was merged. Change subject: engine: Use AuditLogImpl for logging events ...................................................................... engine: Use AuditLogImpl for logging events Instead of instantiating a command only for the sake of logging, the AuditLogImpl can be used instead for the exact purpose. In addition, the exact code was used for logging both host set memory and host set cpu actions. Code was reused for that purpose. Change-Id: I15fb72bfeb747b71ef5ede0026b26696ee68c025 Signed-off-by: Moti Asayag <masayag@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java 1 file changed, 14 insertions(+), 26 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/77223 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15fb72bfeb747b71ef5ede0026b26696ee68c025 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>