----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50565/#review143950 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py (line 65) <https://reviews.apache.org/r/50565/#comment209894> I don't think you need the `upgrade_type is not None` since you have an equality check right after it. ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py (lines 66 - 67) <https://reviews.apache.org/r/50565/#comment209896> I'd rather see a specific check in the stack feature stuff for this; it's not clear what combination of stack features affect this. ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_usersync.py (line 74) <https://reviews.apache.org/r/50565/#comment209895> I don't think you need the `upgrade_type is not None` since you have an equality check right after it. - Jonathan Hurley On July 28, 2016, 9:18 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50565/ > ----------------------------------------------------------- > > (Updated July 28, 2016, 9:18 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan > Hurley, Nate Cole, Srimanth Gunturi, and Velmurugan Periasamy. > > > Bugs: AMBARI-17945 > https://issues.apache.org/jira/browse/AMBARI-17945 > > > Repository: ambari > > > Description > ------- > > As part of changes done in this AMBARI-17840, following changes are required > to handle EU scenario : > > 1] need to change the owner/mode of stop script and > 2] create symbolic link if it does not exists. > > > This patch handles the following upgrade scenarios: > 1) ranger admin from 2.2.x to 2.2.y > 2) ranger usersync from 2.2.x to 2.3, 2.4 > 3) ranger kms from 2.3 to 2.3.x, 2.4, 2.5 > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py > c0534f3 > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_usersync.py > 5ce0617 > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms_server.py > cbdcc8c > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms_service.py > 55c0486 > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/params.py > 4888aa6 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/config-upgrade.xml > 95310d5 > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.3.xml > 6e7e272 > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.4.xml > b368db3 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml > 9eb4a44 > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.4.xml > e67b82c > > Diff: https://reviews.apache.org/r/50565/diff/ > > > Testing > ------- > > Tested EU from > 2.2 to 2.2 > 2.2 to 2.4 > 2.4 to 2.5 > > > Thanks, > > Mugdha Varadkar > >
