> On Oct. 1, 2014, 12:41 p.m., Nate Cole wrote: > >
Thanks for the review. > On Oct. 1, 2014, 12:41 p.m., Nate Cole wrote: > > ambari-server/src/main/python/ambari-server.py, lines 3150-3153 > > <https://reviews.apache.org/r/26194/diff/1/?file=709783#file709783line3150> > > > > Do timeouts eventually get into the 'ERROR' bucket? Yes, a timeout should result in an ERROR status. > On Oct. 1, 2014, 12:41 p.m., Nate Cole wrote: > > ambari-server/src/main/python/ambari-server.py, lines 3173-3175 > > <https://reviews.apache.org/r/26194/diff/1/?file=709783#file709783line3173> > > > > Nit: If this file has garbage or nothing in it, this block may leave > > you with an open file handle. I usually use a "with open(file, 'r') as f:" > > that auto-closes the file object. Good point. I'll make the change. - Tom ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26194/#review55068 ----------------------------------------------------------- On Oct. 1, 2014, 12:43 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26194/ > ----------------------------------------------------------- > > (Updated Oct. 1, 2014, 12:43 p.m.) > > > Review request for Ambari, Jonathan Hurley and Nate Cole. > > > Bugs: AMBARI-7569 > https://issues.apache.org/jira/browse/AMBARI-7569 > > > Repository: ambari > > > Description > ------- > > The command line 'ambari-server sync-ldap' should support the options as > follows ... > > {quote} > "ambari-server sync-ldap --existing" which would would first look at the > ambari database and only clean-up those users and groups. This way, it > doesn't pull any new ones, it a) removes users that are in ambari but out of > ldap, b) remove groups that are in ambari but out of ldap and c) syncs the > group membership of the groups that are in ambari > > "ambari-server sync-ldap --users users.txt --groups groups.txt" which would > allow people to just sync a specific set of users and groups from ldap to > ambari. The .txt files would be the user and group ids to sync into ambari. > This is going to be important for people that just want to bring "a specific > set" of users + groups into ambari from ldap (not the "all"). And then once > this has happened, running with --existing option periodically helps keep > these "in sync" > {quote} > > > Diffs > ----- > > ambari-server/sbin/ambari-server 027bf87 > ambari-server/src/main/python/ambari-server.py 9059319 > ambari-server/src/test/python/TestAmbariServer.py 7f769f1 > > Diff: https://reviews.apache.org/r/26194/diff/ > > > Testing > ------- > > Results : > > Tests run: 2076, Failures: 0, Errors: 0, Skipped: 16 > > ... > > [INFO] Executed tasks > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 33:19.993s > [INFO] Finished at: Tue Sep 30 17:15:25 EDT 2014 > [INFO] Final Memory: 38M/293M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Tom Beerbower > >
