Uploaded image for project: 'Ambari'
  1. Ambari
  2. AMBARI-7191

Alerts: rename old alerts endpoint for new alerts and provide summary info

    Details

      Description

      Currently alerts (from Nagios) are propagated to the UI in the form:

      {
        "href" : "http://localhost:8080/api/v1/clusters/c1/services/HDFS?fields=alerts",
        "ServiceInfo" : {
          "cluster_name" : "c1",
          "service_name" : "HDFS"
        },
        "alerts" : {
          "detail" : [ ... ]
          "summary": {
            "OK": x,
            "WARNING": y,
            "CRITICAL": z
          }
      }
      

      This should be renamed to make way for the alerts endpoint. Summary data should be placed into an alerts_summary element.

      THIS IS AN API BREAKING CHANGE.

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Ambari-trunk-Commit #492 (See https://builds.apache.org/job/Ambari-trunk-Commit/492/)
          AMBARI-7191. Alerts: rename old alerts endpoint for new alerts and provide summary info (ncole) (ncole: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=1f1059cff1ac749e5bdc8fd98febccb7ca346652)

          • ambari-server/src/main/java/org/apache/ambari/server/api/services/HostService.java
          • ambari-server/src/main/java/org/apache/ambari/server/api/resources/AlertResourceDefinition.java
          • ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterService.java
          • ambari-server/src/main/java/org/apache/ambari/server/api/services/AlertService.java
          • ambari-web/app/controllers/main/alerts_controller.js
          • ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java
          • ambari-server/src/main/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProvider.java
          • ambari-web/app/mappers/hosts_mapper.js
          • ambari-server/src/main/java/org/apache/ambari/server/api/services/ServiceService.java
          • ambari-server/src/test/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProviderTest.java
          • ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
          • ambari-web/app/utils/ajax/ajax.js
          • ambari-web/app/mappers/service_mapper.js
          • ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
          • ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertsDAO.java
          • ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertSummaryDTO.java
          • ambari-web/test/controllers/main/alerts_controller_test.js
          • ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertSummaryPropertyProvider.java
          • ambari-web/app/controllers/global/update_controller.js
          • ambari-web/app/controllers/main/host.js
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Ambari-trunk-Commit #492 (See https://builds.apache.org/job/Ambari-trunk-Commit/492/ ) AMBARI-7191 . Alerts: rename old alerts endpoint for new alerts and provide summary info (ncole) (ncole: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=1f1059cff1ac749e5bdc8fd98febccb7ca346652 ) ambari-server/src/main/java/org/apache/ambari/server/api/services/HostService.java ambari-server/src/main/java/org/apache/ambari/server/api/resources/AlertResourceDefinition.java ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterService.java ambari-server/src/main/java/org/apache/ambari/server/api/services/AlertService.java ambari-web/app/controllers/main/alerts_controller.js ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java ambari-server/src/main/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProvider.java ambari-web/app/mappers/hosts_mapper.js ambari-server/src/main/java/org/apache/ambari/server/api/services/ServiceService.java ambari-server/src/test/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProviderTest.java ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java ambari-web/app/utils/ajax/ajax.js ambari-web/app/mappers/service_mapper.js ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertsDAO.java ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertSummaryDTO.java ambari-web/test/controllers/main/alerts_controller_test.js ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertSummaryPropertyProvider.java ambari-web/app/controllers/global/update_controller.js ambari-web/app/controllers/main/host.js
          Hide
          ncole@hortonworks.com Nate Cole added a comment -

          commit 1f1059cff1ac749e5bdc8fd98febccb7ca346652

          Show
          ncole@hortonworks.com Nate Cole added a comment - commit 1f1059cff1ac749e5bdc8fd98febccb7ca346652

            People

            • Assignee:
              ncole@hortonworks.com Nate Cole
              Reporter:
              ncole@hortonworks.com Nate Cole
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development