-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29062/
-----------------------------------------------------------
Review request for Ambari and Nate Cole.
Bugs: AMBARI-8723
https://issues.apache.org/jira/browse/AMBARI-8723
Repository: ambari
Description
-------
Configured cluster with two DataNodes. Modified the percent DNs available alert
to be 10% for warning, 99% for crit. Shutdown one DN and crit alert is fired.
Should be warning since only 50% of DNs are affected (not 99%).
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/events/AlertDefinitionChangedEvent.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/events/AlertDefinitionDisabledEvent.java
2fe3ba9
ambari-server/src/main/java/org/apache/ambari/server/events/AmbariEvent.java
04217f1
ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertAggregateListener.java
6d8f34a
ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertLifecycleListener.java
df95d3d
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAO.java
23de17e
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertsDAO.java
47d26cb
ambari-server/src/main/java/org/apache/ambari/server/state/alert/Source.java
09ddeb3
ambari-server/src/main/java/org/apache/ambari/server/state/services/AlertNoticeDispatchService.java
69f3393
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java
31e93e4
ambari-server/src/test/java/org/apache/ambari/server/state/alerts/AlertEventPublisherTest.java
271c078
Diff: https://reviews.apache.org/r/29062/diff/
Testing
-------
Manual tests between 0, 33%, 50% and 100% for WARNING/CRITICAl aggregate
definitions. New tests added.
Thanks,
Jonathan Hurley
