You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Dmytro Sen <ds...@hortonworks.com> on 2016/02/23 22:27:31 UTC

Review Request 43900: Implement AMS collector certificate verification on monitor and service check sides

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43900/
-----------------------------------------------------------

Review request for Ambari, Aravindan Vijayan and Sid Wagle.


Bugs: AMBARI-15148
    https://issues.apache.org/jira/browse/AMBARI-15148


Repository: ambari


Description
-------

Implement AMS collector certificate verification on monitor and service check sides.
+Grafana
Implement http(-s) connetion reusing to reduce processing overhead for SSL handshaking


Diffs
-----

  ambari-metrics/ambari-metrics-host-monitoring/conf/unix/metric_monitor.ini 5952982 
  ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/config_reader.py d533537 
  ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/controller.py 1713501 
  ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/emitter.py 4e39ab5 
  ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/security.py PRE-CREATION 
  ambari-metrics/ambari-metrics-host-monitoring/src/test/python/core/TestEmitter.py 8f5236a 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py 417574b 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/functions.py 140c24c 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/metrics_grafana_util.py 02caa11 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/network.py PRE-CREATION 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/params.py f1ff998 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py 4346f0f 
  ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/templates/metric_monitor.ini.j2 4e2d0f5 
  ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py 64b16c6 
  ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_grafana.py 755bb4f 
  ambari-server/src/test/python/stacks/2.0.6/configs/default.json adf12ad 

Diff: https://reviews.apache.org/r/43900/diff/


Testing
-------

Unit tests passed


Thanks,

Dmytro Sen


Re: Review Request 43900: Implement AMS collector certificate verification on monitor and service check sides

Posted by Sid Wagle <sw...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43900/#review120396
-----------------------------------------------------------


Ship it!




Ship It!

- Sid Wagle


On Feb. 23, 2016, 9:27 p.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43900/
> -----------------------------------------------------------
> 
> (Updated Feb. 23, 2016, 9:27 p.m.)
> 
> 
> Review request for Ambari, Aravindan Vijayan and Sid Wagle.
> 
> 
> Bugs: AMBARI-15148
>     https://issues.apache.org/jira/browse/AMBARI-15148
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Implement AMS collector certificate verification on monitor and service check sides.
> +Grafana
> Implement http(-s) connetion reusing to reduce processing overhead for SSL handshaking
> 
> 
> Diffs
> -----
> 
>   ambari-metrics/ambari-metrics-host-monitoring/conf/unix/metric_monitor.ini 5952982 
>   ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/config_reader.py d533537 
>   ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/controller.py 1713501 
>   ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/emitter.py 4e39ab5 
>   ambari-metrics/ambari-metrics-host-monitoring/src/main/python/core/security.py PRE-CREATION 
>   ambari-metrics/ambari-metrics-host-monitoring/src/test/python/core/TestEmitter.py 8f5236a 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py 417574b 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/functions.py 140c24c 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/metrics_grafana_util.py 02caa11 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/network.py PRE-CREATION 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/params.py f1ff998 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py 4346f0f 
>   ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/templates/metric_monitor.ini.j2 4e2d0f5 
>   ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_collector.py 64b16c6 
>   ambari-server/src/test/python/stacks/2.0.6/AMBARI_METRICS/test_metrics_grafana.py 755bb4f 
>   ambari-server/src/test/python/stacks/2.0.6/configs/default.json adf12ad 
> 
> Diff: https://reviews.apache.org/r/43900/diff/
> 
> 
> Testing
> -------
> 
> Unit tests passed
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>