-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30312/
-----------------------------------------------------------
Review request for Ambari, Myroslav Papirkovskyy and Sid Wagle.
Bugs: AMBARI-9348
https://issues.apache.org/jira/browse/AMBARI-9348
Repository: ambari
Description
-------
AMS Collector and monitor don't respect customized properties
ams_collector_pid_dir
ams_collector_log_dir
ams_monitor_pid_dir
ams_monitor_log_dir
Diffs
-----
ambari-metrics/ambari-metrics-assembly/pom.xml 6236187
ambari-metrics/ambari-metrics-timelineservice/conf/unix/ambari-metrics-collector
4d6eed7
ambari-metrics/ambari-metrics-timelineservice/pom.xml 46504a7
ambari-server/src/main/resources/common-services/AMS/0.1.0/configuration/ams-env.xml
48fffb3
ambari-server/src/main/resources/common-services/AMS/0.1.0/package/alerts/alert_ambari_metrics_monitor.py
5841267
Diff: https://reviews.apache.org/r/30312/diff/
Testing
-------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Main ....................................... SUCCESS [2.729s]
[INFO] Apache Ambari Project POM ......................... SUCCESS [0.287s]
[INFO] Ambari Web ........................................ SUCCESS [1:13.868s]
[INFO] Ambari Views ...................................... SUCCESS [4.585s]
[INFO] Ambari Admin View ................................. SUCCESS [9.827s]
[INFO] Ambari Server ..................................... SUCCESS [38:24.418s]
[INFO] Ambari Agent ...................................... SUCCESS [16.681s]
[INFO] Ambari Client ..................................... SUCCESS [0.031s]
[INFO] Ambari Python Client .............................. SUCCESS [0.251s]
[INFO] Ambari Groovy Client .............................. SUCCESS [9.482s]
[INFO] Ambari Shell ...................................... SUCCESS [0.030s]
[INFO] Ambari Python Shell ............................... SUCCESS [0.036s]
[INFO] Ambari Groovy Shell ............................... SUCCESS [6.066s]
Thanks,
Dmytro Sen