Change in ovirt-engine[master]: core: Fix empty quota events tab

lhornyak at redhat.com lhornyak at redhat.com
Wed Mar 27 13:35:57 UTC 2013


Laszlo Hornyak has submitted this change and it was merged.

Change subject: core: Fix empty quota events tab
......................................................................


core: Fix empty quota events tab

The Events sub tab under the quota always remained empty.

The reasons for that were:
 a. The quota id was never saved in the audit_log table in the DB
 b. The search query for quota events was parsed wrong

Added the quota ID and name to the AuditLogableBase.
Added correct parsing for the search to recognize 'quota' as an object.

Change-Id: I207db27df01705660dc33874cd0e222028c0ff63
Bug-Url: https://bugzilla.redhat.com/920226
Signed-off-by: Ofri Masad <omasad at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/quota/QuotaManager.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/quota/QuotaManagerAuditLogger.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogableBase.java
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/AuditCrossRefAutoCompleter.java
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SearchObjectAutoCompleter.java
6 files changed, 42 insertions(+), 0 deletions(-)

Approvals:
  Laszlo Hornyak: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I207db27df01705660dc33874cd0e222028c0ff63
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <omasad at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak at redhat.com>
Gerrit-Reviewer: ofri masad <omasad at redhat.com>



More information about the Engine-commits mailing list