Change in ovirt-engine[master]: gluster: improved gluster audit log util

sanjal at redhat.com sanjal at redhat.com
Mon Mar 25 10:41:12 UTC 2013


Shireesh Anjal has submitted this change and it was merged.

Change subject: gluster: improved gluster audit log util
......................................................................


gluster: improved gluster audit log util

Create object of AuditLogableBase and set appropriate properties
on it instead of overriding methods.

Refactoring in AuditLogableBase:
- Set vds id inside setVds
- New setter for glusterVolume

Change-Id: I52212ddb694d727248182d32abc1e1f5a21f76ee
Bug-Url: https://bugzilla.redhat.com/916112
Signed-off-by: Shireesh Anjal <sanjal at redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/gluster/GlusterAuditLogUtil.java
2 files changed, 17 insertions(+), 32 deletions(-)

Approvals:
  Shireesh Anjal: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/13001
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I52212ddb694d727248182d32abc1e1f5a21f76ee
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list