----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49309/#review139904 -----------------------------------------------------------
Fix it, then Ship it! ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py (line 273) <https://reviews.apache.org/r/49309/#comment205244> Should assign a default value since can still be None. ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py (line 275) <https://reviews.apache.org/r/49309/#comment205245> Shouldn't access index 1 without first checking the length ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py (line 280) <https://reviews.apache.org/r/49309/#comment205246> this is equivalent to just custom_log4j = has_logsearch - Alejandro Fernandez On June 28, 2016, 11:47 a.m., Mugdha Varadkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49309/ > ----------------------------------------------------------- > > (Updated June 28, 2016, 11:47 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Don Bosco Durai, Gautam > Borad, Oliver Szabo, Srimanth Gunturi, and Velmurugan Periasamy. > > > Bugs: AMBARI-17462 > https://issues.apache.org/jira/browse/AMBARI-17462 > > > Repository: ambari > > > Description > ------- > > From Ambari installation, install Log search client package with Ranger > Service for stack 2.5 onwards > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > c46309e > > ambari-common/src/main/python/resource_management/libraries/functions/package_conditions.py > ecc4f49 > > ambari-common/src/main/python/resource_management/libraries/functions/solr_cloud_util.py > bab65dc > ambari-server/src/main/resources/common-services/RANGER/0.4.0/metainfo.xml > 40ad69f > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py > 817d3f7 > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_admin.py > f48720c > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/setup_ranger_xml.py > e1b5bc5 > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/templates/ranger_solr_jass_conf.j2 > PRE-CREATION > > ambari-server/src/main/resources/common-services/RANGER/0.6.0/configuration/ranger-env.xml > eff0cd2 > ambari-server/src/main/resources/common-services/RANGER/0.6.0/metainfo.xml > adfb8ff > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > 789c5ba > ambari-server/src/test/python/stacks/2.5/RANGER/test_ranger_admin.py > 9b3c631 > ambari-server/src/test/python/stacks/2.5/configs/ranger-admin-default.json > 53da262 > ambari-server/src/test/python/stacks/2.5/configs/ranger-admin-secured.json > 14571bd > > Diff: https://reviews.apache.org/r/49309/diff/ > > > Testing > ------- > > Running tests for stack:2.5 service:RANGER > test_configure_default (test_ranger_admin.TestRangerAdmin) ... ok > test_configure_secured (test_ranger_admin.TestRangerAdmin) ... ok > test_start_default (test_ranger_admin.TestRangerAdmin) ... ok > test_start_secured (test_ranger_admin.TestRangerAdmin) ... ok > test_stop_default (test_ranger_admin.TestRangerAdmin) ... ok > test_configure_default (test_ranger_usersync.TestRangerUsersync) ... ok > test_configure_secured (test_ranger_usersync.TestRangerUsersync) ... ok > test_start_default (test_ranger_usersync.TestRangerUsersync) ... ok > test_stop_default (test_ranger_usersync.TestRangerUsersync) ... ok > test_configure_default (test_ranger_tagsync.TestRangerTagsync) ... ok > test_configure_secured (test_ranger_tagsync.TestRangerTagsync) ... ok > test_start_default (test_ranger_tagsync.TestRangerTagsync) ... ok > test_stop_default (test_ranger_tagsync.TestRangerTagsync) ... ok > > ---------------------------------------------------------------------- > Ran 13 tests in 0.330s > > OK > > > Thanks, > > Mugdha Varadkar > >
