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


Ship it!




Ship It!

- Dmytro Grinenko


On July 9, 2016, 2:08 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49856/
> -----------------------------------------------------------
> 
> (Updated July 9, 2016, 2:08 p.m.)
> 
> 
> Review request for Ambari and Dmytro Grinenko.
> 
> 
> Bugs: AMBARI-17638
>     https://issues.apache.org/jira/browse/AMBARI-17638
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1\. Dir should be created if not exists  
> 2\. This file seems to relevant only to the RM
> 
>     
>     
>     File 
> "/var/lib/ambari-agent/cache/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py",
>  line 336, in yarn 
>     group=params.user_group 
>     File "/usr/lib/python2.6/site-packages/resource_management/core/base.py", 
> line 154, in __init__ 
>     self.env.run() 
>     File 
> "/usr/lib/python2.6/site-packages/resource_management/core/environment.py", 
> line 160, in run 
>     self.run_action(resource, action) 
>     File 
> "/usr/lib/python2.6/site-packages/resource_management/core/environment.py", 
> line 124, in run_action 
>     provider_action() 
>     File 
> "/usr/lib/python2.6/site-packages/resource_management/core/providers/system.py",
>  line 87, in action_create 
>     raise Fail("Applying %s failed, parent directory %s doesn't exist" % 
> (self.resource, dirname)) 
>     resource_management.core.exceptions.Fail: Applying 
> File['/etc/horton/exclude/yarn.exclude'] failed, parent directory 
> /etc/horton/exclude doesn't exist
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
>  f6bfae8 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py
>  b6771c1 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py 
> e4d48b8 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py 
> edd9615 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_nodemanager.py 3afea7a 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_resourcemanager.py 
> aacf7e0 
>   ambari-server/src/test/python/stacks/2.0.6/YARN/test_yarn_client.py fe033c6 
>   ambari-server/src/test/python/stacks/2.1/YARN/test_apptimelineserver.py 
> 9674960 
>   ambari-server/src/test/python/stacks/2.3/YARN/test_ats_1_5.py c382b20 
> 
> Diff: https://reviews.apache.org/r/49856/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>

Reply via email to