-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52455/
-----------------------------------------------------------
(Updated Oct. 4, 2016, 4:38 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 (updated)
-----
ambari-admin/src/main/resources/ui/admin-web/app/index.html b3f1bd1
ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/groups/GroupsEditCtrl.js
2e7830b
ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/groups/GroupsListCtrl.js
1f86393
ambari-admin/src/main/resources/ui/admin-web/app/scripts/i18n.config.js
af22d7f
ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/Group.js
660306c
ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/GroupConstants.js
PRE-CREATION
ambari-admin/src/main/resources/ui/admin-web/app/scripts/services/UserConstants.js
da63c6d
ambari-admin/src/main/resources/ui/admin-web/app/views/groups/edit.html
8271432
ambari-admin/src/main/resources/ui/admin-web/app/views/groups/list.html
b359ede
ambari-admin/src/main/resources/ui/admin-web/app/views/users/show.html
e576b28
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 (updated)
----------------
New patch
https://reviews.apache.org/media/uploaded/files/2016/10/03/460d7193-8d55-4ce3-96c6-ad3240502e7b__AMBARI-18476.patch
Final patch
https://reviews.apache.org/media/uploaded/files/2016/10/04/3d025c3b-1f60-4a0c-9a49-0802a21c99fd__AMBARI-18476.patch
Thanks,
Sangeeta Ravindran
