Change in ovirt-engine[master]: engine, webadmin: Clear all option in RHEV admin portal does...

oourfali at redhat.com oourfali at redhat.com
Tue Jun 23 08:48:44 UTC 2015


Oved Ourfali has submitted this change and it was merged.

Change subject: engine, webadmin: Clear all option in RHEV admin portal does not clear any alerts
......................................................................


engine, webadmin: Clear all option in RHEV admin portal does not clear any alerts

Rename "Clear All" to "Display All" and implement
"Clear All" that clears all alerts and events in the
footer panel.

Change-Id: I07e78feaf972b0a3097c64b592a28e71256d9805
Bug-Url: https://bugzilla.redhat.com/1206264
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClearAllAuditLogAlertsCommand.java
C backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClearAllAuditLogEventsCommand.java
D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClearDismissedAuditLogAlertsCommand.java
D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ClearDismissedAuditLogEventsCommand.java
C backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DisplayAllAuditLogAlertsCommand.java
C backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DisplayAllAuditLogEventsCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAO.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAODbFacadeImpl.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendEventsResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendEventsResourceTest.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/events/AlertListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/events/EventListModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/footer/AlertsEventsFooterView.java
M packaging/dbscripts/audit_log_sp.sql
17 files changed, 139 insertions(+), 143 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, but someone else must approve
  Ravi Nori: Verified
  Oved Ourfali: Looks good to me, approved; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I07e78feaf972b0a3097c64b592a28e71256d9805
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list