----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53738/#review155850 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Nov. 14, 2016, 3:54 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53738/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2016, 3:54 p.m.) > > > Review request for Ambari, Nate Cole and Swapan Shridhar. > > > Bugs: AMBARI-18887 > https://issues.apache.org/jira/browse/AMBARI-18887 > > > Repository: ambari > > > Description > ------- > > - Ambari 2.4.0.0 -> Ambari 2.4.2.0-115 > - HDP 2.5.0 -> HDP 2.5.3.0.-14 > > While upgrading HDP from 2.5.0.0 to HDP 2.5.3.0-14, hive service check > failed with below error > > {code} > Traceback (most recent call last): > File > "/var/lib/ambari-agent/cache/common-services/HIVE/0.12.0.2.0/package/scripts/service_check.py", > line 191, in <module> > HiveServiceCheck().execute() > File > "/usr/lib/python2.6/site-packages/resource_management/libraries/script/script.py", > line 280, in execute > method(env) > File > "/var/lib/ambari-agent/cache/common-services/HIVE/0.12.0.2.0/package/scripts/service_check.py", > line 88, in service_check > params.hive_http_endpoint) > File > "/var/lib/ambari-agent/cache/common-services/HIVE/0.12.0.2.0/package/scripts/service_check.py", > line 173, in check_llap > exec_path = os.environ['PATH'] + os.pathsep + params.hadoop_bin_dir + > os.pathsep + upgrade_hive_bin > NameError: global name 'os' is not defined > {code} > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py > 3caaefc > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/service_check.py > 90a97fe > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_service_check.py > 30b4eb7 > > Diff: https://reviews.apache.org/r/53738/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Jonathan Hurley > >
