You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Jaimin Jetly <ja...@hortonworks.com> on 2015/01/06 03:18:59 UTC
Re: Review Request 29517: Zookeeper service components should indicate
security state
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29517/#review66777
-----------------------------------------------------------
Ship it!
Ship It!
- Jaimin Jetly
On Dec. 31, 2014, 6:14 p.m., Robert Levas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29517/
> -----------------------------------------------------------
>
> (Updated Dec. 31, 2014, 6:14 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Sid Wagle, and Yusaku Sako.
>
>
> Bugs: AMBARI-8797
> https://issues.apache.org/jira/browse/AMBARI-8797
>
>
> Repository: ambari
>
>
> Description
> -------
>
> The Zookeeper service components should indicate security state when queried by Ambari Agent via STATUS_COMMAND.
>
>
> Diffs
> -----
>
> ambari-common/src/main/python/resource_management/libraries/functions/security_commons.py 535a53b
> ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/status_params.py 36c5c30
> ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper_server.py 27cdfc1
> ambari-server/src/test/python/stacks/2.0.6/ZOOKEEPER/test_zookeeper_server.py ff3b413
>
> Diff: https://reviews.apache.org/r/29517/diff/
>
>
> Testing
> -------
>
> Manually tested in test cluster
>
> #Jenkins Test Results
>
> OK
> Running tests for stack:2.0.6 service:ZOOKEEPER
> test_service_check_22 (test_zookeeper_service_check.TestServiceCheck) ... ok
> test_service_check_default (test_zookeeper_service_check.TestServiceCheck) ... ok
> test_service_check_secured (test_zookeeper_service_check.TestServiceCheck) ... ok
> test_configure_default (test_zookeeper_client.TestZookeeperClient) ... ok
> test_configure_secured (test_zookeeper_client.TestZookeeperClient) ... ok
> test_configure_default (test_zookeeper_server.TestZookeeperServer) ... ok
> test_configure_secured (test_zookeeper_server.TestZookeeperServer) ... ok
> **test_security_status (test_zookeeper_server.TestZookeeperServer) ... ok**
> test_start_default (test_zookeeper_server.TestZookeeperServer) ... ok
> test_start_secured (test_zookeeper_server.TestZookeeperServer) ... ok
> test_stop_default (test_zookeeper_server.TestZookeeperServer) ... ok
> test_stop_secured (test_zookeeper_server.TestZookeeperServer) ... ok
>
> ----------------------------------------------------------------------
> Ran 12 tests in 0.223s
>
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 01:08 h
> [INFO] Finished at: 2014-12-31T18:00:17+00:00
> [INFO] Final Memory: 43M/541M
> [INFO] ------------------------------------------------------------------------
>
>
> Thanks,
>
> Robert Levas
>
>