> On May 18, 2016, 10:27 a.m., Andrew Onischuk wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py, > > line 56 > > <https://reviews.apache.org/r/47529/diff/1/?file=1386595#file1386595line56> > > > > Can we monitor self.status until the stop is done. Or skip the check in > > script.py. > > > > This we be a major slownload for 'Stop all' considering it takes ~40s > > as of now.
Another solution is to skip that check in script.py if service_name=="RANGER". If that is acceptable to stop afterwards Ambari stop command. - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47529/#review133721 ----------------------------------------------------------- On May 18, 2016, 10:22 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47529/ > ----------------------------------------------------------- > > (Updated May 18, 2016, 10:22 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Gautam > Borad, Jayush Luniya, and Velmurugan Periasamy. > > > Bugs: AMBARI-16727 > https://issues.apache.org/jira/browse/AMBARI-16727 > > > Repository: ambari > > > Description > ------- > > Adding sleep time of 15 seconds after execution of stop command as Ranger > Admin process requires time to stop. Once the logic for pid is available from > Ranger code, will update the code. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/package_conditions.py > 1ffbbde > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py > c015367 > > Diff: https://reviews.apache.org/r/47529/diff/ > > > Testing > ------- > > Tested Ranger on centos 6 > > > Thanks, > > Mugdha Varadkar > >