Change in ovirt-engine[ovirt-engine-4.0]: engine: prevent notification flood for number of LVs

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: engine: prevent notification flood for number of LVs
......................................................................


engine: prevent notification flood for number of LVs

When creating more than <AlertOnNumberOfLVs> LVs on a storage domain
(300 by default), we log that the number of LVs on the storage domain
exceeded the threshold.

Since one can register to get this notification by mail, we should
prevent a mail flood.
This patch restricts this event to be logged once in 3 hours for each
storage domain.

Change-Id: I4df27e5b7211cb00c49221e99d4cdeab82f21bd9
Bug-Url: https://bugzilla.redhat.com/1275182
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/storage/StorageDomainHelper.java
2 files changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4df27e5b7211cb00c49221e99d4cdeab82f21bd9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>