----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59113/#review174407 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/python/ambari_server/setupMpacks.py Lines 722 (patched) <https://reviews.apache.org/r/59113/#comment247542> can we use lowercase - Alejandro Fernandez On May 10, 2017, 12:21 a.m., Madhuvanthi Radhakrishnan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59113/ > ----------------------------------------------------------- > > (Updated May 10, 2017, 12:21 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit > Mohanty. > > > Bugs: AMBARI-20974 > https://issues.apache.org/jira/browse/AMBARI-20974 > > > Repository: ambari > > > Description > ------- > > fix to update permissions after installing an mpack > > > Diffs > ----- > > ambari-server/src/main/python/ambari_server/setupMpacks.py 167f264b8f > ambari-server/src/test/python/TestMpacks.py ef8194502e > > > Diff: https://reviews.apache.org/r/59113/diff/2/ > > > Testing > ------- > > Unit Testing: > Total run:1170 > 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: 02:15 min > [INFO] Finished at: 2017-05-09T14:30:43-07:00 > [INFO] Final Memory: 94M/1439M > > Functional Testing: > Install Ambari-2.5.1 using non-root ambari admin user > Install HDP-2.6 (HDFS, Yarn, ZKFS, Storm, Kafka) > Install HDF-3.0 > ambari-server restart > Add Nifi and start service. > Verify that the permissions are set correctly in > /var/lib/ambari-server/resources/mpacks, > /var/lib/ambari-server/resources/extensions, > /var/lib/ambari-server/common-services/NIFI, > /var/lib/ambari-server/stacks/HDP/2.6/services/NIFI. > > > Thanks, > > Madhuvanthi Radhakrishnan > >