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