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

oourfali at redhat.com oourfali at redhat.com
Wed Aug 27 07:23:51 UTC 2014


Oved Ourfali 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>
---
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/31996
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie831e8ec49b81bcb49f106f2fb377cff4dc4566b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list