----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29670/#review67094 -----------------------------------------------------------
Ship it! This isn't obvious, but there is a class that can be used to help find these issues: `org.apache.ambari.server.stack.KerberosDescriptorTest`. But there would be work do to make sure the configurations are set and not ignored. I will look into generically testing this scenario. - Robert Levas On Jan. 7, 2015, 4:13 p.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29670/ > ----------------------------------------------------------- > > (Updated Jan. 7, 2015, 4:13 p.m.) > > > Review request for Ambari, Robert Levas, Srimanth Gunturi, and Yusaku Sako. > > > Bugs: AMBARI-9033 > https://issues.apache.org/jira/browse/AMBARI-9033 > > > Repository: ambari > > > Description > ------- > > This was due to yarn.nodemanager.container-executor.class not updating > correctly due to incorrect kerberos descriptor format. > The patch apart from fixing this issue also places > yarn.nodemanager.container-executor.class property in a correct category > "Isolation" > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/kerberos.json > d6aaa6e > ambari-web/app/data/HDP2.2/yarn_properties.js 401ff08 > > Diff: https://reviews.apache.org/r/29670/diff/ > > > Testing > ------- > > tested e2e. > ambari-web unit test result: > 5208 tests complete (9 seconds) > 60 tests pending > > > Thanks, > > Jaimin Jetly > >
