----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59851/#review177073 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On June 6, 2017, 6:44 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59851/ > ----------------------------------------------------------- > > (Updated June 6, 2017, 6:44 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Nate Cole. > > > Bugs: AMBARI-21188 > https://issues.apache.org/jira/browse/AMBARI-21188 > > > Repository: ambari > > > Description > ------- > > The configuration symlinks for a component after installation should reflect > something similar to the following: > > /etc/component/conf -> /usr/hdp/current/component/conf > /usr/hdp/current/component/conf -> /etc/component/<version>/conf > > For example: > {noformat} > [root@c6403 ~]# ll /etc/zookeeper/ > total 12 > drwxr-xr-x 3 root root 4096 Jun 5 20:19 2.4.2.0-236 > drwxr-xr-x 3 root root 4096 Jun 5 20:38 2.6.0.0-334 > lrwxrwxrwx 1 root root 26 Jun 5 20:38 conf -> > /usr/hdp/current/zookeeper-server/conf > drwxr-xr-x 2 zookeeper hadoop 4096 Jun 5 20:17 conf.backup > > [root@c6403 ~]# ll /usr/hdp/current/zookeeper-server/conf > lrwxrwxrwx 1 root root 28 Jun 5 20:38 /usr/hdp/current/zookeeper-server/conf > -> /etc/zookeeper/2.6.0.0-334/0 > {noformat} > > This is the way that the structure exists after a normal installation today. > However, it seems that distribution a new stack breaks this: > > {code} > [root@c6403 zookeeper]# ll /etc/zookeeper/ > total 12 > drwxr-xr-x 3 root root 4096 Jun 5 20:19 2.4.2.0-236 > drwxr-xr-x 3 root root 4096 Jun 5 20:38 2.6.0.0-334 > lrwxrwxrwx 1 root root 26 Jun 5 20:38 conf -> > /etc/zookeeper/conf.backup > drwxr-xr-x 2 zookeeper hadoop 4096 Jun 5 20:17 conf.backup > {code} > > The {{conf}} symlink is now pointing to {{conf.backup}} which is the interim > temporary location. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/conf_select.py > facf186f0e > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > c12dff16b1 > > > Diff: https://reviews.apache.org/r/59851/diff/1/ > > > Testing > ------- > > Total run:1168 > Total errors:0 > Total failures:0 > OK > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 01:23 min > [INFO] Finished at: 2017-06-06T14:43:58-04:00 > [INFO] Final Memory: 18M/245M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Jonathan Hurley > >
