Change in ovirt-engine[master]: engine: Use AuditLogableImpl for vm logging

Code Review gerrit at ovirt.org
Sun May 7 15:58:55 UTC 2017


>From Moti Asayag <masayag at redhat.com>:

Moti Asayag has submitted this change and it was merged.

Change subject: engine: Use AuditLogableImpl for vm logging
......................................................................


engine: Use AuditLogableImpl for vm logging

With AuditLogableImpl, the properties to be logged can be set while
creating the event log, without any need to access the DB and populate
missing attributes. Since all of the properties for logging host and vm
entities are already available, prefer using this method.

Change-Id: I63fb6c437e5ab035ff3ca4a62ae2e3ffac9207f9
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClearNonResponsiveVdsVmsCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java
2 files changed, 8 insertions(+), 7 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Moti Asayag: Verified



-- 
To view, visit https://gerrit.ovirt.org/76449
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I63fb6c437e5ab035ff3ca4a62ae2e3ffac9207f9
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika 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