----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46345/#review129431 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On April 18, 2016, 8:13 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46345/ > ----------------------------------------------------------- > > (Updated April 18, 2016, 8:13 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Nate Cole. > > > Bugs: AMBARI-15950 > https://issues.apache.org/jira/browse/AMBARI-15950 > > > Repository: ambari > > > Description > ------- > > STR: > > - Install cluster on HDP 2.3 > - Add a host with DataNode after cluster has been installed > > For hosts part of the cluster creation: > {code} > [root@os-r6-orydgs-upg-sanity-211-5 /]# ls -la /etc/hadoop/ > total 24 > drwxr-xr-x. 4 root root 4096 Apr 18 04:49 . > drwxr-xr-x. 114 root root 12288 Apr 18 10:53 .. > drwxr-xr-x. 3 root root 4096 Apr 18 04:49 2.3.4.0-3485 > lrwxrwxrwx. 1 root root 35 Apr 18 04:49 conf -> > /usr/hdp/current/hadoop-client/conf > drwxr-xr-x. 2 root root 4096 Apr 18 04:49 conf.install > {code} > > For hosts was added after cluster creation: > {code} > [root@os-r6-orydgs-upg-sanity-211-2 ~]# ls -la /etc/hadoop > total 16 > drwxr-xr-x. 4 root root 4096 Apr 18 09:34 . > drwxr-xr-x. 80 root root 4096 Apr 18 09:35 .. > drwxr-xr-x. 3 root root 4096 Apr 18 09:34 2.3.4.0-3485 > lrwxrwxrwx. 1 root root 23 Apr 18 09:34 conf -> /etc/hadoop/conf.backup > drwxr-xr-x. 2 root root 4096 Apr 18 10:36 conf.backup > {code} > > This causes the {{/etc/<component>/conf}} folder to point indefinitely to the > {{/etc/<component>/conf.backup}}. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/conf_select.py > 7ce4a6f > > Diff: https://reviews.apache.org/r/46345/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Jonathan Hurley > >