----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57204/#review167548 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java Lines 457-458 (patched) <https://reviews.apache.org/r/57204/#comment239427> There is no need to worry about the existing value in the service's configration. The idea is to sync up the user-defined Kerberos Descriptor with the stack-level Kerberos Descriptor. Any existing service configuration will be untouched, and by removing the relelvant configuration specification from the user-defined Kerberos descriptor, the config will remain untouched during Kerberos-related operations, as desired. - Robert Levas On March 1, 2017, 11:12 a.m., Eugene Chekanskiy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57204/ > ----------------------------------------------------------- > > (Updated March 1, 2017, 11:12 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Robert Levas, and Vitalyi > Brodetskyi. > > > Bugs: AMBARI-20237 > https://issues.apache.org/jira/browse/AMBARI-20237 > > > Repository: ambari > > > Description > ------- > > STR > 1. Deployed cluster with Ambari version: 2.4.1 and HDP version: 2.5.0(secure > cluster) > 2. Upgrade Ambari to 2.5 > 3. Regenerate keytabs for missing hosts and let it complete > 4. Go to YARN - configs > Result > Observed that yarn.nodemanager.linux-container-executor.cgroups.mount-path > got added as a property with blank value > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java > 39a129d > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java > 64536cb > > ambari-server/src/test/resources/kerberos/test_kerberos_descriptor_2_5_infra_solr.json > 172ad05 > > > Diff: https://reviews.apache.org/r/57204/diff/1/ > > > Testing > ------- > > [INFO] Reactor Summary: > [INFO] > [INFO] Ambari Main ....................................... SUCCESS [1.412s] > [INFO] Apache Ambari Project POM ......................... SUCCESS [0.063s] > [INFO] Ambari Web ........................................ SUCCESS [2:43.041s] > [INFO] Ambari Views ...................................... SUCCESS [2.060s] > [INFO] Ambari Admin View ................................. SUCCESS [53.373s] > [INFO] utility ........................................... SUCCESS [0.964s] > [INFO] ambari-metrics .................................... SUCCESS [0.232s] > [INFO] Ambari Metrics Common ............................. SUCCESS [6.656s] > [INFO] Ambari Server ..................................... SUCCESS > [26:02.995s] > [INFO] Ambari Agent ...................................... SUCCESS [1:23.748s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 31:15.402s > [INFO] Finished at: Wed Mar 01 17:58:13 EET 2017 > [INFO] Final Memory: 242M/2931M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Eugene Chekanskiy > >
