-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28874/
-----------------------------------------------------------
Review request for Ambari, Srimanth Gunturi and Tom Beerbower.
Bugs: AMBARI-8623
https://issues.apache.org/jira/browse/AMBARI-8623
Repository: ambari
Description
-------
In Ambari-1.7.0 the cluster endpoint used to have a summary of host-counts by
severity. We need such a field in Ambari-2.0.0.
{code}
alerts: {
summary: {
CRITICAL: 1,
OK: 4,
PASSIVE: 0,
WARNING: 0
}
}
{code}
The alerts_summary structure currently returned by the 2.0.0 API are the
summary counts of alerts in the system, not those of the hosts. There are
alerts that don't have a host (such as aggregate alerts).
Add a new field to the cluster endpoint; something like {{alerts_summary_hosts
}}
{noformat}
http://localhost:8080/api/v1/clusters/c1?fields=alerts_summary_hosts
{
"href" :
"http://localhost:8080/api/v1/clusters/c1?fields=alerts_summary_hosts";,
"Clusters" : {
"cluster_name" : "c1",
"version" : "HDP-2.2"
},
"alerts_summary_hosts" : {
"CRITICAL" : 1,
"OK" : 41,
"UNKNOWN" : 0,
"WARNING" : 0
}
}
{noformat}
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertSummaryPropertyProvider.java
e9fef0e
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertsDAO.java
4908eea
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java
6876877
Diff: https://reviews.apache.org/r/28874/diff/
Testing
-------
New tests added to cover new summary.
Thanks,
Jonathan Hurley
