----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45141/#review124708 -----------------------------------------------------------
Note: To add new patch click on Update>Update Patch and upload the patch. - Jayush Luniya On March 22, 2016, 1:10 a.m., Juanjo Marron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45141/ > ----------------------------------------------------------- > > (Updated March 22, 2016, 1:10 a.m.) > > > Review request for Ambari, Alejandro Fernandez and Jayush Luniya. > > > Bugs: AMBARI-14435 > https://issues.apache.org/jira/browse/AMBARI-14435 > > > Repository: ambari > > > Description > ------- > > Apply the stack featurization prototype detailed on AMBARI-13364 to > ZOOKEEPER service plus the comments and reviews received on: > https://reviews.apache.org/r/44933/ > > This patch removes the hardcoded stack versions in ZK common-services code > (centralized version and parameterized stack configurations will be faced in > other JIRAS) > > stack_features is now a property in .json format in > HDP/2.0.6/properties/stack_features.json > With the structure: > { > "stack_features": [ > { "name": "feature1", "description" : "Feature1 support", "min_version" : > "Y.Y.Y.Y" , "max_version" : "X.X.X.X"} > , > ... > ] > } > where min_version/max_version are optional constraints. > > The upgrade constants, such has ROLLING_UPGRADE = "rolling_upgrade" has been > added to in a new StackFeature in > resource_management/libraries/functions/constants.py > > It is added as part of the HDP stack configurations on > /HDP/2.0.6/configuration/cluster-env.xml, introducing a new stack_features > property > > New resource_management/libraries/functions/stack_fetaures.py has been > introduced to parse the json file and called from service code to check if > the stack supports the required feature. > > The stack version used in params_linux.py comparison is > stack_version_formatted = hostLevelParams\stack_version > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > 0adaef9 > > ambari-common/src/main/python/resource_management/libraries/functions/stack_features.py > PRE-CREATION > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/params_linux.py > 9ff9125 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/status_params.py > 9ae8440 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper.py > 5712ce4 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper_client.py > 25ace24 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper_server.py > d5b6898 > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5.2.0/package/scripts/zookeeper_service.py > 6fe0772 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml > 5f77f48 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/45141/diff/ > > > Testing > ------- > > Zookeeper fresh installation on top of latest trunk > > > Thanks, > > Juanjo Marron > >