-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27884/
-----------------------------------------------------------

(Updated Nov. 11, 2014, 4:41 p.m.)


Review request for Ambari, Nate Cole, Oleg Nechiporenko, Srimanth Gunturi, and 
Tom Beerbower.


Bugs: AMBARI-8289
    https://issues.apache.org/jira/browse/AMBARI-8289


Repository: ambari


Description (updated)
-------

Add ability to get summary info about alerts grouped by 
{{alert_definition_name}}.

```
http://localhost:8080/api/v1/clusters/c1/alerts?format=groupedSummary
{
  "href" : 
"http://localhost:8080/api/v1/clusters/c1/alerts?format=groupedSummary";,
  "alerts_summary_grouped" : [
    {
      "definition_id" : 34,
      "definition_name" : "yarn_resourcemanager_webui",
      "summary" : {
        "OK" : {
          "count" : 1,
          "original_timestamp" : 1415735183450
        },
        "WARNING" : {
          "count" : 0,
          "original_timestamp" : 0
        },
        "CRITICAL" : {
          "count" : 0,
          "original_timestamp" : 0
        },
        "UNKNOWN" : {
          "count" : 0,
          "original_timestamp" : 0
        }
      }
    },
    ...
  ]
}
```


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/api/query/render/AlertSummaryGroupedRenderer.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/api/query/render/AlertSummaryRenderer.java
 afe9798 
  
ambari-server/src/main/java/org/apache/ambari/server/api/resources/AlertResourceDefinition.java
 18f206e 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertResourceProvider.java
 3430f8d 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AlertResourceProviderTest.java
 2bac86a 

Diff: https://reviews.apache.org/r/27884/diff/


Testing
-------

New test added to cover the new renderer.


Thanks,

Jonathan Hurley

Reply via email to