> On Oct. 3, 2016, 2:34 p.m., Alexandr Antonenko wrote: > > ambari-admin/src/main/resources/ui/admin-web/app/index.html, line 162 > > <https://reviews.apache.org/r/52455/diff/1/?file=1517321#file1517321line162> > > > > I don't see this newly created file in diff files list. You were able > > to build UI without this files ? UI should fail upon building because of > > declared file that does not exist
Hi Aleksandr, I accidentally missed adding the file when I created the patch. I have now added it. New patch has been uplaoded. - Sangeeta ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52455/#review151170 ----------------------------------------------------------- On Oct. 3, 2016, 4:35 p.m., Sangeeta Ravindran wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52455/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2016, 4:35 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Alexandr Antonenko, and Di Li. > > > Bugs: AMBARI-18476 > https://issues.apache.org/jira/browse/AMBARI-18476 > > > Repository: ambari > > > Description > ------- > > AMBARI-12263 adds support for PAM as authentication mechanism for accessing > Ambari UI/REST. The changes in this review request cover the corresponding > changes in the amabri-admin code. > Since a new column groupt_type has been added for groups, the UI will display > labels for group type and enable/disable group delete/add member > functionality based on the group_type instead of the ldap_group flag. > Patch includes similar changes for users. The user_type will be used to > determine if the user can be deleted or if the user's password can be changed. > > > Diffs > ----- > > > ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/GroupConstants.js > PRE-CREATION > > Diff: https://reviews.apache.org/r/52455/diff/ > > > Testing > ------- > > Manual testing. > Tests in ambari admin project ran clean. > PhantomJS 1.9.7 (Linux): Executed 86 of 86 SUCCESS (0.771 secs / 0.75 secs) > > > File Attachments > ---------------- > > New patch > > https://reviews.apache.org/media/uploaded/files/2016/10/03/460d7193-8d55-4ce3-96c6-ad3240502e7b__AMBARI-18476.patch > > > Thanks, > > Sangeeta Ravindran > >
