-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36519/
-----------------------------------------------------------
(Updated Aug. 10, 2015, 9:52 p.m.)
Review request for Ambari and Alejandro Fernandez.
Changes
-------
renamed variable "dir" to "parent_dir"
Bugs: AMBARI-12349
https://issues.apache.org/jira/browse/AMBARI-12349
Repository: ambari
Description
-------
When configure hadoop from install wizard, set "File that stores mount point"
or "net.topology.script.file.name" to a non default location, e.g
/etc/hadoop1/conf/dfs_data_dir_mount.hist
/etc/hadoop1/conf/topology_script.py
install failed because datanode could not be start, with error message
Applying File['/etc/hadoop1/conf/topology_mappings.data'] failed, parent
directory /etc/hadoop1/conf doesn't exist
This is because rack_awareness.py only handles the creation of the file but not
the parent directory. Default value doesn't have this problem because the
directory /etc/hadoop/conf was created in yum install.
Diffs (updated)
-----
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/rack_awareness.py
6b0bff6
ambari-server/src/test/python/stacks/2.0.6/hooks/before-START/test_before_start.py
df39d2f
Diff: https://reviews.apache.org/r/36519/diff/
Testing
-------
unit test: test_hook_refresh_topology_custom_directories
(test_before_start.TestHookBeforeStart) ... ok
Thanks,
Di Li