-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29788/
-----------------------------------------------------------

Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, and Nate Cole.


Bugs: AMBARI-8917
    https://issues.apache.org/jira/browse/AMBARI-8917


Repository: ambari


Description
-------

The prepare_rolling_restart() functions call copy_tarballs_to_hdfs(), which 
gets the HDP version from the first component that matches the regex in the 
output of hdp-select, which is incorrect.
Instead of getting the first component, it should query a specific component, 
e.g., "hdp-select status hiveserver2"


Diffs
-----

  
ambari-common/src/main/python/resource_management/libraries/functions/dynamic_variable_interpretation.py
 4f0dad3b308d325ec358a63c895e4708a6403943 
  ambari-server/src/main/python/ambari-server.py 
af27f5f3be7bf15597fb4545fcd7e393096ca4a7 
  
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/namenode_upgrade.py
 d80d37755bece5e172e75535f3c9a5d3a1fa9b6c 
  
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server.py
 055916b2b26ca6a8413a8f38317e5381f3d50ddb 
  
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/webhcat.py
 9d53ea90b9e7e732a0ff0b8728cb12e898d3a4ac 
  
ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/service_check.py
 a0e04abbe6c2eaede2ed8efbc79026bbc9d97135 
  
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/historyserver.py
 57bd37963047a7456d57a87f2032cd00a5cf8f9b 
  ambari-server/src/test/python/stacks/2.0.6/HIVE/test_hive_server.py 
5dcc016b2fffd8af9334ecba7e342108ae0a8116 

Diff: https://reviews.apache.org/r/29788/diff/


Testing
-------

Copied changed scripts to a new cluster, and restarted HiveServer2 and 
HistoryServer to make sure that the logs showed hdp-select using that component.


Thanks,

Alejandro Fernandez

Reply via email to