Change in ovirt-engine[ovirt-engine-3.5]: core: Do not suppress dismissed alerts

tnisan at redhat.com tnisan at redhat.com
Wed Aug 27 08:42:44 UTC 2014


Tal Nisan has submitted this change and it was merged.

Change subject: core: Do not suppress dismissed alerts
......................................................................


core: Do not suppress dismissed alerts

No health check alert is issued in ui after first message was issued

This patch fixed a problem when an alert is dismissed by the user.
Current functionality did not addressed dismissed alerts and show for
each host instance one alert from each type.
In the case that the alert was dismissed by the user, it will not show
again.
This patch fix the SP that insert to audit log not to suppress the alert
in case that the alert exists for the host instance but it was dismissed
by the user (delete=true in audit_log table)

Change-Id: Ie831e8ec49b81bcb49f106f2fb377cff4dc4566b
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1133611
Signed-off-by: Eli Mesika <emesika at redhat.com>
(cherry picked from commit 57cb88b9b75e705504fd738dbcc1b9d4eac092c3)
---
M packaging/dbscripts/audit_log_sp.sql
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Eli Mesika: Verified
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie831e8ec49b81bcb49f106f2fb377cff4dc4566b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list