----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29958/#review68427 -----------------------------------------------------------
Ship it! I noticed that you did a data back up but no restore. Just wanted to verify that's correct. ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params.py <https://reviews.apache.org/r/29958/#comment112640> Does Knox have a /usr/hdp/current/knox-client directory like many of the other services do? If so, we probably need to determine which directory to use based on the command. If not, then disregard. ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/upgrade.py <https://reviews.apache.org/r/29958/#comment112641> Is only the data directory needed in the backup? Some components also need the conf directory as well. - Jonathan Hurley On Jan. 15, 2015, 11:10 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29958/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2015, 11:10 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Nate > Cole. > > > Bugs: AMBARI-9169 > https://issues.apache.org/jira/browse/AMBARI-9169 > > > Repository: ambari > > > Description > ------- > > Create the upgrade pack and steps for upgrading Knox Servers. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox_gateway.py > 9332bff > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params.py > 3c91992 > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/upgrade.py > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml > 15aa1f5 > ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py debd77e > > Diff: https://reviews.apache.org/r/29958/diff/ > > > Testing > ------- > > Unit test added. All existing tests pass. > > Manual testing is in progress... > > > Thanks, > > Tom Beerbower > >
