> On Oct. 11, 2017, 2:08 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py > > Lines 201-205 (original), 201-207 (patched) > > <https://reviews.apache.org/r/62890/diff/1/?file=1852401#file1852401line201> > > > > Why not just use the upgrade_summary module for all of this - would > > protect you against changes in the future.
Updated in latest patch - Mugdha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62890/#review187657 ----------------------------------------------------------- On Oct. 12, 2017, 9 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62890/ > ----------------------------------------------------------- > > (Updated Oct. 12, 2017, 9 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan > Hurley, Nate Cole, and Velmurugan Periasamy. > > > Bugs: AMBARI-22204 > https://issues.apache.org/jira/browse/AMBARI-22204 > > > Repository: ambari > > > Description > ------- > > After AMBARI-21581 changes, need to update Ranger Admin pre-upgrade task. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py > f779c18 > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_tagsync.py > 74439bf > > ambari-server/src/main/resources/stacks/BigInsights/4.2/services/RANGER/package/scripts/ranger_admin.py > 37555cc > > > Diff: https://reviews.apache.org/r/62890/diff/2/ > > > Testing > ------- > > Tested EU & RU for Ranger Admin. > > > Thanks, > > Mugdha Varadkar > >
