[Hawkular-commits] [hawkular/hawkular] dd1218: Hawkular-464 Deployment Failure Event Alerting

Mike Thompson mtho11 at users.noreply.github.com
Fri Nov 20 15:16:58 EST 2015


  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular
  Commit: dd12188cc1d224b9bc40e1fb7bb6baab483624ce
      https://github.com/hawkular/hawkular/commit/dd12188cc1d224b9bc40e1fb7bb6baab483624ce
  Author: Jay Shaughnessy <jshaughn at redhat.com>
  Date:   2015-11-20 (Fri, 20 Nov 2015)

  Changed paths:
    M console/src/main/scripts/plugins/metrics/html/directives/alert.html
    M console/src/main/scripts/plugins/metrics/html/triggers/availability.html
    A console/src/main/scripts/plugins/metrics/html/triggers/event.html
    M console/src/main/scripts/plugins/metrics/ts/alertsCenterTriggerList.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDatasourcesDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDeploymentsDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerJvmDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerWebDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAppServerDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsPlugin.ts
    M console/src/main/scripts/plugins/metrics/ts/services/alertsManager.ts
    A console/src/main/scripts/plugins/metrics/ts/triggers/eventTrigger.ts
    M modules/avail-creator/pom.xml
    M modules/pinger/pom.xml
    M pom.xml

  Log Message:
  -----------
  Hawkular-464 Deployment Failure Event Alerting
Add alert generation for failed deployments.
- New src-deps versions for component enhancements to support this
  - with mazz, enhanced bus-commons with better classloading/deserialization support for MDBs
  - with mazz, enhanced command-gateway to support multiple commands on ws responses, now can notify UI and put a response on the bus
  - enhanced alerts to consume ws deploy app response messages on bus and generate events
- Added new trigger definition on app-servers to fire on a failed deployment event
  - note, also storing successful deployment events for future use/display
- Added new UI support for 'Event' trigger-type and DEPLOYMENT_FAIL alert-type

Also:
- fixed time-window bug in getAlerts query for some trigger types
- make bus-commons dep provided (by nest) for pinger
- make bus-commons dep provided (by nest) for avail-creator
- up src revision slightly for accounts to get the longer grace period for c* connection


  Commit: 593d868ce6b90c3009e83e8cc31e930fe3b262a9
      https://github.com/hawkular/hawkular/commit/593d868ce6b90c3009e83e8cc31e930fe3b262a9
  Author: Jay Shaughnessy <jshaughn at redhat.com>
  Date:   2015-11-20 (Fri, 20 Nov 2015)

  Changed paths:
    M console/src/main/scripts/plugins/metrics/ts/alertsCenterTriggerList.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDatasourcesDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDeploymentsDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerJvmDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerPlatformDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerWebDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAvailability.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsResponse.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsTypes.ts

  Log Message:
  -----------
  PR feedback
- convert some growing if-else blocks to switch statements
- have controller's getAlerts() impls type more variables
- add types to a couple of other places.
Also:
- delete getAlerts from platform controller. It's unused and out of date.
  (Is this controller even used at all?)


  Commit: e73ed8900f910107d6243c55b2779f380e5e26a7
      https://github.com/hawkular/hawkular/commit/e73ed8900f910107d6243c55b2779f380e5e26a7
  Author: Jay Shaughnessy <jshaughn at redhat.com>
  Date:   2015-11-20 (Fri, 20 Nov 2015)

  Changed paths:
    M console/src/main/scripts/plugins/metrics/html/app-details/detail-deployments.html
    M console/src/main/scripts/plugins/metrics/html/app-details/detail-jvm.html
    M console/src/main/scripts/plugins/metrics/html/app-details/detail-web.html
    M console/src/main/scripts/plugins/metrics/html/url-alerts.html
    M console/src/main/scripts/plugins/metrics/html/url-availability.html
    M console/src/main/scripts/plugins/metrics/html/url-response-time.html
    M console/src/main/scripts/plugins/metrics/ts/alertsCenterTriggerList.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDeploymentsDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerJvmDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerWebDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAlerts.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAppServerDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAvailability.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsResponse.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsTypes.ts
    M console/src/main/scripts/plugins/metrics/ts/services/alertsManager.ts

  Log Message:
  -----------
  PR Feedback
- use this.$routeParams.resourceId for clarity. Note, it may still be set to
  a local var in certain code blocks, when repeated use lessens readability.
-
Also:
- fix bug with alert count display for URL avail and metric tab views
- for consistency, change 'Alert Settings' to 'Definitions' in url tab views
- remove some leftover debug console logging


  Commit: 56db44b0bf75abd2beb025e3c3d844d481868a0e
      https://github.com/hawkular/hawkular/commit/56db44b0bf75abd2beb025e3c3d844d481868a0e
  Author: Mike Thompson <mtho11 at users.noreply.github.com>
  Date:   2015-11-20 (Fri, 20 Nov 2015)

  Changed paths:
    M console/src/main/scripts/plugins/metrics/html/app-details/detail-deployments.html
    M console/src/main/scripts/plugins/metrics/html/app-details/detail-jvm.html
    M console/src/main/scripts/plugins/metrics/html/app-details/detail-web.html
    M console/src/main/scripts/plugins/metrics/html/directives/alert.html
    M console/src/main/scripts/plugins/metrics/html/triggers/availability.html
    A console/src/main/scripts/plugins/metrics/html/triggers/event.html
    M console/src/main/scripts/plugins/metrics/html/url-alerts.html
    M console/src/main/scripts/plugins/metrics/html/url-availability.html
    M console/src/main/scripts/plugins/metrics/html/url-response-time.html
    M console/src/main/scripts/plugins/metrics/ts/alertsCenterTriggerList.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDatasourcesDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerDeploymentsDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerJvmDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerPlatformDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/app-details/appServerWebDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAlerts.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAppServerDetails.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsAvailability.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsPlugin.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsResponse.ts
    M console/src/main/scripts/plugins/metrics/ts/metricsTypes.ts
    M console/src/main/scripts/plugins/metrics/ts/services/alertsManager.ts
    A console/src/main/scripts/plugins/metrics/ts/triggers/eventTrigger.ts
    M modules/avail-creator/pom.xml
    M modules/pinger/pom.xml
    M pom.xml

  Log Message:
  -----------
  Merge pull request #675 from jshaughn/hawkular-464

Hawkular-464 Deployment Failure Event Alerting


Compare: https://github.com/hawkular/hawkular/compare/a93a5761b3a6...56db44b0bf75


More information about the hawkular-commits mailing list