Change in ovirt-engine[master]: 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: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Idan Shaby: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4df27e5b7211cb00c49221e99d4cdeab82f21bd9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>