----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49856/ -----------------------------------------------------------
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