Re: Review Request 47417: Replace '*' to hdp version in lzo packages

[Vitalyi Brodetskyi]

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

(Updated Травень 17, 2016, 10:24 до полудня)


Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, and Sumit 
Mohanty.


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


Repository: ambari


Description
-------

As i understood(according to discussion with Andrew O) we had some jira for '*' 
to hdp version replacement in all packages. But here it was not replaced:
{code}
def get_lzo_packages(stack_version_unformatted):
  lzo_packages = []
 
  if OSCheck.is_redhat_family() or OSCheck.is_suse_family():
    lzo_packages += ["lzo", "hadoop-lzo-native"]
  elif OSCheck.is_ubuntu_family():
    lzo_packages += ["liblzo2-2"]

  if stack_version_unformatted and 
check_stack_feature(StackFeature.ROLLING_UPGRADE, stack_version_unformatted):
    lzo_packages += ["hadooplzo_*"]
  else:
    lzo_packages += ["hadoop-lzo"]

  return lzo_packages
{code}


Diffs (updated)
-----

  
ambari-common/src/main/python/resource_management/libraries/functions/get_lzo_packages.py
 e189d62 
  
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/install_params.py
 fe488c3 
  
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py
 784da9c 
  
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py
 75924c5 
  ambari-server/src/test/python/stacks/2.0.6/configs/client-upgrade.json 
47ab0a3 
  ambari-server/src/test/python/stacks/2.0.6/configs/default.json f0e5208 
  ambari-server/src/test/python/stacks/2.0.6/configs/nn_ru_lzo.json d44b002 
  ambari-server/src/test/python/stacks/2.0.6/configs/ranger-namenode-start.json 
b163a61 
  
ambari-server/src/test/python/stacks/2.0.6/hooks/after-INSTALL/test_after_install.py
 6c7fe18 
  ambari-server/src/test/python/stacks/2.1/HIVE/test_hive_metastore.py c36b428 
  
ambari-server/src/test/python/stacks/2.2/configs/journalnode-upgrade-hdfs-secure.json
 7b07d87 
  ambari-server/src/test/python/stacks/2.2/configs/journalnode-upgrade.json 
cf84bb7 
  ambari-server/src/test/python/stacks/2.2/configs/oozie-downgrade.json 7e5346c 
  ambari-server/src/test/python/stacks/2.2/configs/oozie-upgrade.json 1c75d65 

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


Testing
-------

mvn clean test


Thanks,

Vitalyi Brodetskyi