> On Dec. 12, 2017, 6:54 p.m., Robert Levas wrote: > > This is looking really good. Upgrade logic to the appropriate > > UpgradeCatalog class.
Upgrade will be seperate patch. I want to finish server and agent side changes to implement upgrade after all tables and data are stable. > On Dec. 12, 2017, 6:54 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatProcessor.java > > Lines 456-464 (patched) > > <https://reviews.apache.org/r/64118/diff/3/?file=1909894#file1909894line464> > > > > Did you intend to leave this here, or will it be fixed before this is > > committed? Most likely it will be removed after agent side refactoring. On final stage of refactoring all TODOs will be reviewed and removed. - Eugene ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64118/#review193576 ----------------------------------------------------------- On Dec. 13, 2017, 6:14 p.m., Eugene Chekanskiy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64118/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2017, 6:14 p.m.) > > > Review request for Ambari, Attila Magyar, Balázs Bence Sári, and Robert Levas. > > > Bugs: AMBARI-22530 > https://issues.apache.org/jira/browse/AMBARI-22530 > > > Repository: ambari > > > Description > ------- > > Moved out of data files for principal generation. > Now before every kerberos action all info written to database. > > Server actions determine what principals to process based on filters passed > to action, this allow: > > * selective generation > * split generation process per-host/per-service/per-component basis and make > generation concurrent for seperate parts > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java > 53cceb0ded > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatProcessor.java > 83d2c9808d > > ambari-server/src/main/java/org/apache/ambari/server/controller/DeleteIdentityHandler.java > a7b9d80df0 > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java > 749943dc00 > > ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java > ab85aa1d7c > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostKerberosIdentityResourceProvider.java > 52ab9b56ae > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/KerberosKeytabDAO.java > a8723b7bfa > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/KerberosKeytabPrincipalDAO.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/KerberosPrincipalDAO.java > 81e4b3d0bf > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/KerberosPrincipalHostDAO.java > f27dc48ea4 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostGroupComponentEntityPK.java > 0898133bb5 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/KerberosKeytabEntity.java > a25931b946 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/KerberosKeytabPrincipalEntity.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/KerberosKeytabServiceMappingEntity.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/KerberosPrincipalEntity.java > 5dd54ca30c > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/KerberosPrincipalHostEntity.java > d4e80c65d2 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/KerberosPrincipalHostEntityPK.java > 7e57e4ae95 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/AbstractPrepareKerberosServerAction.java > b8affb4e19 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CleanupServerAction.java > 002076d85c > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/ConfigureAmbariIdentitiesServerAction.java > 338415280f > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreateKeytabFilesServerAction.java > 5ec4c1011e > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/CreatePrincipalsServerAction.java > 0c906592af > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/DestroyPrincipalsServerAction.java > 4c80bd425e > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/FinalizeKerberosServerAction.java > bfd5e4036d > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/KerberosServerAction.java > ff5f5cef1c > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareEnableKerberosServerAction.java > 671ad95c8f > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/PrepareKerberosIdentitiesServerAction.java > 83a2106afd > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/stageutils/KerberosKeytabController.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/stageutils/ResolvedKerberosKeytab.java > 17e484ad10 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/kerberos/stageutils/ResolvedKerberosPrincipal.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/PreconfigureKerberosAction.java > ca78dbb8a2 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java > 5ac1ac316d > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql 7045240b30 > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql c950c7ef83 > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 537ae196c5 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql b4952c2e86 > ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql > 4fb0d0981a > ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 8a88aba905 > ambari-server/src/main/resources/META-INF/persistence.xml 686c8312cd > > ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java > b4ff5c10ab > > ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java > ee87d24d8a > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostKerberosIdentityResourceProviderTest.java > 59fbba0eac > ambari-server/src/test/java/org/apache/ambari/server/orm/db/DDLTests.java > 96cf64e53c > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/ConfigureAmbariIdentitiesServerActionTest.java > 39dee24ad4 > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/FinalizeKerberosServerActionTest.java > c9301f3b7e > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/KerberosServerActionTest.java > e6f0868360 > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/PreconfigureKerberosActionTest.java > a08f7a0a25 > > > Diff: https://reviews.apache.org/r/64118/diff/4/ > > > Testing > ------- > > manual testing, all kind of regenerate keytabs > mvn clean test > > > Thanks, > > Eugene Chekanskiy > >
