----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55853/#review162827 -----------------------------------------------------------
Ship it! Ship It! - Sumit Mohanty On Jan. 23, 2017, 8:47 p.m., Nahappan Somasundaram wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55853/ > ----------------------------------------------------------- > > (Updated Jan. 23, 2017, 8:47 p.m.) > > > Review request for Ambari, Miklos Gergely and Sumit Mohanty. > > > Bugs: AMBARI-19681 > https://issues.apache.org/jira/browse/AMBARI-19681 > > > Repository: ambari > > > Description > ------- > > AMBARI-19681: Credential store should add hadoop credential provider path > property to all affected configuration types > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py > 02f42123b9b392251e1315f930d63cb11c0de2c7 > > Diff: https://reviews.apache.org/r/55853/diff/ > > > Testing > ------- > > ** 1. mvn test -DskipSurefireTests ** > > ---------------------------------------------------------------------- > Ran 270 tests in 6.740s > > OK > ---------------------------------------------------------------------- > Total run:1151 > Total errors:0 > Total failures:0 > OK > [INFO] > [INFO] --- maven-checkstyle-plugin:2.17:check (checkstyle) @ ambari-server --- > [INFO] Starting audit... > Audit done. > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 1:13.542s > [INFO] Finished at: Mon Jan 23 11:57:11 PST 2017 > [INFO] Final Memory: 50M/798M > [INFO] > ------------------------------------------------------------------------ > > ** 2. Manual testing ** > Deployed the changes to a VM and manually verified that when the command > contains multiple configurations that contain credentials, each config type > gets a *hadoop.security.credential.provider.path* property pointing to its > JCEKS provider. > > > Thanks, > > Nahappan Somasundaram > >
