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




ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_advisor.py (line 
84)
<https://reviews.apache.org/r/47063/#comment196050>

    You should rename this variable (yarn_cgroups_enabled) because it can be 
misleading


- Oliver Szabo


On May 6, 2016, 10:04 a.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47063/
> -----------------------------------------------------------
> 
> (Updated May 6, 2016, 10:04 a.m.)
> 
> 
> Review request for Ambari, Robert Levas, Sumit Mohanty, and Vitalyi 
> Brodetskyi.
> 
> 
> Bugs: AMBARI-16286
>     https://issues.apache.org/jira/browse/AMBARI-16286
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1. Install ambari cluster with few services 
> 2. enable kerberos
> 3. run service check on services
> Expected: all service checks should pass successfully
> Actual: MR service check fails.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/configuration/yarn-env.xml
>  2184c0e 
>   ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_advisor.py 
> d6bc713 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py 
> cf0990d 
>   ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py 
> c48b2c7 
> 
> Diff: https://reviews.apache.org/r/47063/diff/
> 
> 
> Testing
> -------
> 
> Unit tests passed
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>

Reply via email to