----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34750/#review85554 -----------------------------------------------------------
Ship it! Ship It! - Jonathan Hurley On May 27, 2015, 10:58 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34750/ > ----------------------------------------------------------- > > (Updated May 27, 2015, 10:58 p.m.) > > > Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Jonathan > Hurley, and Nate Cole. > > > Bugs: AMBARI-11476 > https://issues.apache.org/jira/browse/AMBARI-11476 > > > Repository: ambari > > > Description > ------- > > This occurs because when Core Masters are upgraded and the round of Service > Checks are ran, the Oozie Service Check does not contain yarn-site.xml in its > config folder. > When Oozie Server is then upgraded, it fails because the tarball of the old > conf folder actually zips the conf folder itself, so when it is unpacked, it > creates /usr/hdp/2.3.0.0-2130/oozie/conf/conf/ > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py > acb1e9f > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py > 47361c4 > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/service_check.py > 11f4431 > ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_service_check.py > 219c874 > > Diff: https://reviews.apache.org/r/34750/diff/ > > > Testing > ------- > > Performed an RU from Dal to Dal+ with Oozie, and all of the service checks > and Oozie Server restart passed. > > ---------------------------------------------------------------------- > Total run:743 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >