Change in ovirt-engine[master]: engine: Use AuditLogImpl for logging events

Code Review gerrit at ovirt.org
Fri Jun 9 08:07:54 UTC 2017


>From Moti Asayag <masayag at 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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list