----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55538/#review161649 -----------------------------------------------------------
Please disregard this patch. I'm going to revise this patch to copy and set the acl instead of adding the paths to the configurations. - Nahappan Somasundaram On Jan. 14, 2017, 12:14 p.m., Nahappan Somasundaram wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55538/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2017, 12:14 p.m.) > > > Review request for Ambari, Jonathan Hurley, Nate Cole, and Sumit Mohanty. > > > Bugs: AMBARI-19545 > https://issues.apache.org/jira/browse/AMBARI-19545 > > > Repository: ambari > > > Description > ------- > > AMBARI-19545: Ambari-agent - In HIVE and OOZIE stack scripts, copy JCEKS file > to desired location > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/CustomServiceOrchestrator.py > 02f42123b9b392251e1315f930d63cb11c0de2c7 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive.py > 4ac3f8eedbdf954795417f2ba6a3406ed677fcaf > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py > 21135ea5c65563716dc1fe5db0d533970d9fdb53 > > ambari-server/src/main/resources/common-services/OOZIE/4.2.0.2.3/configuration/oozie-site.xml > 0ff268b47dcb687cbf8a07392f440b82182f8311 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/HIVE/configuration/hive-site.xml > 856941fec6bb8475c786251ad11c84c065be4c1b > > Diff: https://reviews.apache.org/r/55538/diff/ > > > Testing > ------- > > ** 1. Manual testing ** > > Deployed all the modified files to a VM and manually verified that the JCEKS > files were generated at the location specified in the configuration, and that > the ACLs were also applied correctly. > > ** 2. mvn test -Dtest=ServicePropertiesTest -DskipPythonTests ** > > Audit done. > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 36.153s > [INFO] Finished at: Sat Jan 14 11:59:43 PST 2017 > [INFO] Final Memory: 56M/603M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Nahappan Somasundaram > >