-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59956/
-----------------------------------------------------------
Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene Chekanskiy,
Jonathan Hurley, Laszlo Puskas, and Sebastian Toader.
Bugs: AMBARI-21147
https://issues.apache.org/jira/browse/AMBARI-21147
Repository: ambari
Description
-------
Update Database Access Layer to Support New Database Schema for Improved User
Account Management.
* Update `org.apache.ambari.server.orm.entities.UserEntity`
* Update `org.apache.ambari.server.orm.dao.UserDAO`
* Add `org.apache.ambari.server.orm.entities.UserAuthenticationEntity`
* Add `org.apache.ambari.server.orm.dao.UserAuthenticationDAO`
Note: Some changes will be revisited when updating the different authentication
processes to work with the improved user account management code.
Diffs
-----
ambari-server/docs/api/generated/index.html 7ea4297b99
ambari-server/docs/api/generated/swagger.json d7d54a510f
ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java
fb06e6d8a5
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementController.java
807bded873
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
2a9d6c9ee0
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
2f799b70c4
ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java
4fa236225d
ambari-server/src/main/java/org/apache/ambari/server/controller/ResourceProviderFactory.java
391213858e
ambari-server/src/main/java/org/apache/ambari/server/controller/UserRequest.java
40818c8f48
ambari-server/src/main/java/org/apache/ambari/server/controller/UserResponse.java
5afacb70ef
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java
b35b2a8612
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ActiveWidgetLayoutResourceProvider.java
389f0b2bf2
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UserPrivilegeResourceProvider.java
614f7abda1
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UserResourceProvider.java
2ec63dbaeb
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserAuthenticationDAO.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserDAO.java
ce47c4c38c
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UserAuthenticationEntity.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UserEntity.java
9011eaecec
ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AmbariJWTAuthenticationFilter.java
195c55afa5
ambari-server/src/main/java/org/apache/ambari/server/security/authentication/AuthenticationMethodNotAllowedException.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/security/authentication/kerberos/AmbariAuthToLocalUserDetailsService.java
1e4f6ead08
ambari-server/src/main/java/org/apache/ambari/server/security/authentication/kerberos/AmbariKerberosAuthenticationProperties.java
09422e51e3
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilter.java
ce9a79023d
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLdapAuthenticationProvider.java
b7ff297ce5
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLdapAuthoritiesPopulator.java
d38d44c16f
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariLocalUserProvider.java
37d5d49c37
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariPamAuthenticationProvider.java
0fff8c8122
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariUserAuthorizationFilter.java
95e90b3e49
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AuthorizationHelper.java
64d5e6124f
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/User.java
bff1fd2a16
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/UserType.java
aabd368aeb
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/Users.java
9cdde8fe4d
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/internal/AmbariInternalAuthenticationProvider.java
383e8fac87
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/jwt/AuthenticationJwtUserNotFoundException.java
f18af101a2
ambari-server/src/main/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthenticationFilter.java
258e9da303
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java
f413c697d6
ambari-server/src/main/resources/META-INF/persistence.xml e4045ef536
ambari-server/src/test/java/org/apache/ambari/server/configuration/ConfigurationTest.java
1b8de79737
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
e10e4cd57b
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AbstractPrivilegeResourceProviderTest.java
547bba57ad
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ActiveWidgetLayoutResourceProviderTest.java
4dc06b92c8
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/GroupPrivilegeResourceProviderTest.java
36f6a1e2e4
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UserPrivilegeResourceProviderTest.java
9ccbc11529
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UserResourceProviderDBTest.java
c4f0f349fb
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UserResourceProviderTest.java
d298b7f135
ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java
550cc9fe12
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/UserDAOTest.java
05733fa57a
ambari-server/src/test/java/org/apache/ambari/server/security/SecurityHelperImplTest.java
f15f2f5218
ambari-server/src/test/java/org/apache/ambari/server/security/authentication/AmbariJWTAuthenticationFilterTest.java
de5b768863
ambari-server/src/test/java/org/apache/ambari/server/security/authentication/kerberos/AmbariAuthToLocalUserDetailsServiceTest.java
530bf651bb
ambari-server/src/test/java/org/apache/ambari/server/security/authentication/kerberos/AmbariKerberosAuthenticationPropertiesTest.java
eb26cd839b
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilterTest.java
15e243e224
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationProviderDisableUserTest.java
891ab38638
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariLdapAuthenticationProviderForDNWithSpaceTest.java
442414f14d
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariLdapAuthenticationProviderTest.java
4941bc7afb
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariLocalUserProviderTest.java
2362823b30
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariPamAuthenticationProviderTest.java
8faa6ce316
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariUserAuthenticationFilterTest.java
0483b04ec0
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/TestAmbariLdapAuthoritiesPopulator.java
fff39d8bf3
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/TestUsers.java
e29791f19b
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/UsersTest.java
ac91c904ac
ambari-server/src/test/java/org/apache/ambari/server/security/authorization/jwt/JwtAuthenticationFilterTest.java
24f5f88490
ambari-server/src/test/java/org/apache/ambari/server/security/ldap/AmbariLdapDataPopulatorTest.java
63b69277a4
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog240Test.java
f10665825c
Diff: https://reviews.apache.org/r/59956/diff/1/
Testing
-------
Manually tested in cluster
# Local test results:
```
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 34:03 min
[INFO] Finished at: 2017-06-09T17:05:32-04:00
[INFO] Final Memory: 209M/1768M
[INFO] ------------------------------------------------------------------------
```
# Jenkins test results: PENDING
Thanks,
Robert Levas
