> On Dec. 10, 2015, 7:36 a.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/scripts/Ambaripreupload.py, line 134 > > <https://reviews.apache.org/r/41129/diff/1/?file=1156951#file1156951line134> > > > > @Andrew, > > The symbolic link for /usr/hdp/current/hadoop-client doesnt point to > > /usr/hdp/<hdp_version>/hadoop-client > > > > Can you verify that all the path changes are correct? > > > > > > ls -lh hadoop-client > > lrwxrwxrwx 1 root root 28 Dec 10 02:57 hadoop-client -> > > /usr/hdp/2.3.4.0-3426/hadoop
That did not influence the testing as hadoop binary was in $PATH (which most of time will be the case). But good observation anyways let me fix that. - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41129/#review109702 ----------------------------------------------------------- On Dec. 9, 2015, 9:15 a.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41129/ > ----------------------------------------------------------- > > (Updated Dec. 9, 2015, 9:15 a.m.) > > > Review request for Ambari, Myroslav Papirkovskyy and Vitalyi Brodetskyi. > > > Bugs: AMBARI-14285 > https://issues.apache.org/jira/browse/AMBARI-14285 > > > Repository: ambari > > > Description > ------- > > `ambari-server/src/main/resources/scripts/Ambaripreupload.py` is used to > setup hdfd directories > .A variation of this script is needed so that in can be called during RU. > > The scenario is as follows: > > * User installs the new version of HDP > * User manually calls this script to upload the tarballs and Oozie share > libs > * User goes ahead and performs RU > > This script need to take an input as to what the target HDP version is (-v > optional param). > > > Diffs > ----- > > ambari-server/src/main/resources/scripts/Ambaripreupload.py fd72f24 > > Diff: https://reviews.apache.org/r/41129/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
