[Hawkular-commits] [hawkular/hawkular-alerts] 26fc39: Prometheus external alerter

Lucas Ponce ponce.ballesteros at gmail.com
Wed Jun 28 03:53:18 EDT 2017


  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-alerts
  Commit: 26fc39a063633a92063bd1e037e38c521e4d20df
      https://github.com/hawkular/hawkular-alerts/commit/26fc39a063633a92063bd1e037e38c521e4d20df
  Author: Jay Shaughnessy <jshaughn at redhat.com>
  Date:   2017-06-27 (Tue, 27 Jun 2017)

  Changed paths:
    M hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/pom.xml

  Log Message:
  -----------
  Prometheus external alerter
This is an initial alerter that can set up an ExternalCondition to execute
a Promql query and feed the results back as events for matching the
condition.
- There are two manual tests...
  - one tests a standard promql query
  - one tests a query against the special ALERTS timeseries when P8S is
    configured with alert rules.

Some things to think about given this POC:
- Should we use tags or context for the prometheus labels?
- How well will this really work given that prometheus could return
  many metrics for the single expression?
  - should we think about setting source? Data-driven groups here?


  Commit: 8894c3bb5519979099ad4da4bd0a456d4fd18ea0
      https://github.com/hawkular/hawkular-alerts/commit/8894c3bb5519979099ad4da4bd0a456d4fd18ea0
  Author: Jay Shaughnessy <jshaughn at redhat.com>
  Date:   2017-06-27 (Tue, 27 Jun 2017)

  Changed paths:
    M hawkular-alerters/hawkular-elasticsearch-alerter/deployments/hawkular-elasticsearch-alerter-standalone/src/main/webapp/WEB-INF/web.xml
    M hawkular-alerters/hawkular-prometheus-alerter/deployments/hawkular-prometheus-alerter-standalone/src/main/webapp/WEB-INF/web.xml
    M hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/pom.xml
    A hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/src/main/java/org/hawkular/alerter/prometheus/rest/AlerterPrometheusApp.java
    A hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/src/main/java/org/hawkular/alerter/prometheus/rest/Handler.java
    A hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/src/main/java/org/hawkular/alerter/prometheus/rest/Notification.java
    M pom.xml

  Log Message:
  -----------
  Add WebHook endpoint to the prometheus alerter to make hAlerts into a
notification receiver.  Translates notifications into events.

Also:
- remove security-constraints from elastic standalone alerter. I think
  this was just carried over by mistake.
- add some license exclusions (I should have just made this *.*)


  Commit: 8739ab4ff3e29a8fedd02f771cfc3cb88d9698d3
      https://github.com/hawkular/hawkular-alerts/commit/8739ab4ff3e29a8fedd02f771cfc3cb88d9698d3
  Author: Lucas Ponce <ponce.ballesteros at gmail.com>
  Date:   2017-06-28 (Wed, 28 Jun 2017)

  Changed paths:
    M hawkular-alerters/hawkular-elasticsearch-alerter/deployments/hawkular-elasticsearch-alerter-standalone/src/main/webapp/WEB-INF/web.xml
    M hawkular-alerters/hawkular-prometheus-alerter/deployments/hawkular-prometheus-alerter-standalone/src/main/webapp/WEB-INF/web.xml
    M hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/pom.xml
    A hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/src/main/java/org/hawkular/alerter/prometheus/rest/AlerterPrometheusApp.java
    A hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/src/main/java/org/hawkular/alerter/prometheus/rest/Handler.java
    A hawkular-alerters/hawkular-prometheus-alerter/hawkular-prometheus-alerter-lib/src/main/java/org/hawkular/alerter/prometheus/rest/Notification.java
    M pom.xml

  Log Message:
  -----------
  Merge pull request #355 from jshaughn/prom-webhook

Prom webhook


Compare: https://github.com/hawkular/hawkular-alerts/compare/ddc147297d21...8739ab4ff3e2


More information about the hawkular-commits mailing list