----------------------------------------------------------- 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 > >