----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60844/#review180474 -----------------------------------------------------------
Ship it! Juanjo and I combed thru the codes and did not see any red flag. We did not know the group creation codes was pushed into trunk a while back and got confused a bit ;)). - Tuong On July 13, 2017, 4:03 p.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60844/ > ----------------------------------------------------------- > > (Updated July 13, 2017, 4:03 p.m.) > > > Review request for Ambari, Attila Magyar, Anita Jebaraj, Balázs Bence Sári, > Di Li, Eugene Chekanskiy, Juanjo Marron, Laszlo Puskas, Sebastian Toader, > and Tuong. > > > Bugs: AMBARI-19038 > https://issues.apache.org/jira/browse/AMBARI-19038 > > > Repository: ambari > > > Description > ------- > > After PAM has been declared as the authentication mechanism for Ambari > clients (for example, the Ambari Server web-based UI) where previously the > mechanism was LDAP, a migration process is needed to converts the LDAP users > and groups to PAM users and groups. > > This facility may be invoked using: > > ``` > ambari-server migrate-ldap-pam > ``` > > > Diffs > ----- > > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/groups/GroupsEditCtrl.js > c61e71cc33 > ambari-server/pom.xml b1179a4847 > ambari-server/sbin/ambari-server cc1c923a43 > > ambari-server/src/main/java/org/apache/ambari/server/security/authentication/LdapToPamMigrationHelper.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/Users.java > 83edccc04b > ambari-server/src/main/python/ambari-server.py 71459fb2d3 > ambari-server/src/main/python/ambari_server/setupActions.py 142a4d7cc4 > ambari-server/src/main/python/ambari_server/setupSecurity.py 17d10256fc > > > Diff: https://reviews.apache.org/r/60844/diff/1/ > > > Testing > ------- > > Manually tested > > > Thanks, > > Robert Levas > >
