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




ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py
 (line 168)
<https://reviews.apache.org/r/55083/#comment232632>

    If hdfs-site is only used for NN HA parameters then this is good. If there 
are other hdfs-site parameters that Atlas depends on then you may want a 
dependency that if hdfs-site changes then Atlas should be restarted. Actually, 
I would say just go ahead an add the dependency as if user starts to use 
dfs.internal.nameservice to support distcp after NN HA is enabled then you do 
need Atlas restart after that change.



ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
 (line 226)
<https://reviews.apache.org/r/55083/#comment232631>

    Is this indeed limited to HDP-2.6? What happens for HDP-2.5 and user has NN 
HA with Atlas?


- Sumit Mohanty


On Jan. 12, 2017, 1:58 p.m., Vishal Suvagia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55083/
> -----------------------------------------------------------
> 
> (Updated Jan. 12, 2017, 1:58 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jaimin Jetly, 
> Madhan Neethiraj, Mugdha Varadkar, Oliver Szabo, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-19029
>     https://issues.apache.org/jira/browse/AMBARI-19029
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> After enabling Namenode-HA, atlas had a critical alert that "This host-level 
> alert is triggered if the Metadata Server Web UI is unreachable."
> 
> As a fix we need to include hdfs-site.xml in Atlas conf directory classpath 
> when Namenode-HA is enabled.
> 
> 
> Diffs
> -----
> 
>   
> ambari-common/src/main/python/resource_management/libraries/functions/constants.py
>  a633c13 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py
>  a568cb7 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
>  4b7bd3c 
>   
> ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json
>  ca4c9f5 
>   ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py 
> f2fec70 
>   ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py 584763d 
> 
> Diff: https://reviews.apache.org/r/55083/diff/
> 
> 
> Testing
> -------
> 
> Verified fresh install on Cent-OS 6.
> 
> test_service_check (test_service_check.TestAtlasCheck) ... 2017-01-12 
> 19:24:26,791 - Stack Feature Version Info: stack_version=2.3, 
> version=2.2.1.0-2067, current_cluster_version=None -> 2.2.1.0-2067
> 2017-01-12 19:24:26,795 - Using hadoop conf dir: 
> /usr/hdp/current/hadoop-client/conf
> ok
> 2017-01-12 19:24:26,806 - Stack Feature Version Info: stack_version=2.3, 
> version=2.2.1.0-2067, current_cluster_version=None -> 2.2.1.0-2067
> test_service_check_secure (test_service_check.TestAtlasCheck) ... 2017-01-12 
> 19:24:26,809 - Using hadoop conf dir: /usr/hdp/current/hadoop-client/conf
> ok
> 
> test_configure_default (test_atlas_server.TestAtlasServer) ... 2017-01-12 
> 19:24:51,208 - Stack Feature Version Info: stack_version=2.5, 
> version=2.5.0.0-1235, current_cluster_version=None -> 2.5.0.0-1235
> 2017-01-12 19:24:51,212 - Using hadoop conf dir: 
> /usr/hdp/current/hadoop-client/conf
> 2017-01-12 19:24:51,230 - Stack Feature Version Info: stack_version=2.5, 
> version=2.5.0.0-1235, current_cluster_version=None -> 2.5.0.0-1235
> ok
> test_security_status (test_atlas_server.TestAtlasServer) ... 2017-01-12 
> 19:24:51,245 - Stack Feature Version Info: stack_version=2.5, 
> version=2.5.0.0-1235, current_cluster_version=None -> 2.5.0.0-1235
> 2017-01-12 19:24:51,249 - Using hadoop conf dir: 
> /usr/hdp/current/hadoop-client/conf
> ok
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>

Reply via email to