ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <>
Subject Review Request 24979: Alerts: persist and allow configuration substitution variables for alert definitions
Date Fri, 22 Aug 2014 13:30:56 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for Ambari and Jonathan Hurley.

Bugs: AMBARI-6983

Repository: ambari


* Made some changes from last review - adding constants for source types, save all definitions
in one file instead of one-per-cluster.
* Added substitution variables for those sources that support it.


  ambari-agent/src/main/python/ambari_agent/ 10fdef7 
  ambari-agent/src/main/python/ambari_agent/ 23c28ae 
  ambari-agent/src/main/python/ambari_agent/ fb41759 
  ambari-agent/src/main/python/ambari_agent/alerts/ 6e99692 
  ambari-agent/src/main/python/ambari_agent/alerts/ 2f051c8 
  ambari-agent/src/test/python/ambari_agent/ 0d0563a 
  ambari-agent/src/test/python/ambari_agent/ 9ec23db 
  ambari-agent/src/test/python/ambari_agent/dummy_files/alert_definitions.def 45fb8d0 
  ambari-agent/src/test/python/ambari_agent/dummy_files/definitions.json PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/agent/ 3c9615f

  ambari-server/src/main/java/org/apache/ambari/server/agent/ 80cad4e 
  ambari-server/src/main/java/org/apache/ambari/server/agent/ 65b7b6f

  ambari-server/src/main/java/org/apache/ambari/server/agent/ 5336694



Wanted to get this patch in before being unavailable for a week, so full test suite not made
for this patch.  Verified existing tests are not broken.

[INFO] ------------------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 5.748s
[INFO] Finished at: Fri Aug 22 09:29:41 EDT 2014
[INFO] Final Memory: 7M/81M
[INFO] ------------------------------------------------------------------------


Nate Cole

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message