Change in ovirt-engine[master]: engine: Don't expect any pre-set values for the Logged event

Code Review gerrit at ovirt.org
Sun Apr 30 10:16:49 UTC 2017


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

Moti Asayag has submitted this change and it was merged.

Change subject: engine: Don't expect any pre-set values for the Logged event
......................................................................


engine: Don't expect any pre-set values for the Logged event

With AuditLogableImpl there shouldn't be any assumption about initial
values set to the logged entities.

Therefore, we shouldn't rely on the existence of Host and Cluster when
constructing the key to represent the event.

Change-Id: I90012a2c531ea96728ecb5029fd095f6288e3217
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/EventKeyComposer.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Moti Asayag: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I90012a2c531ea96728ecb5029fd095f6288e3217
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list