> On March 18, 2016, 9:25 p.m., Jayush Luniya wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_fetaures.py, > > line 1 > > <https://reviews.apache.org/r/44933/diff/1/?file=1301827#file1301827line1> > > > > Minor, fix typo in file name > > Juanjo Marron wrote: > OK.Thanks. I'll solve it in the next patch replacing this one
Fixed in: https://reviews.apache.org/r/45141 > On March 18, 2016, 9:25 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/configuration/cluster-env.xml, > > lines 29-35 > > <https://reviews.apache.org/r/44933/diff/1/?file=1301836#file1301836line29> > > > > Dont need to provide min_version if the feature was not enabled at from > > a maint release onwards. Just HDP/2.0.6/configuration/cluster-env.xml will include stack_features property > On March 18, 2016, 9:25 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml, > > line 25 > > <https://reviews.apache.org/r/44933/diff/1/?file=1301834#file1301834line25> > > > > FYI, in https://reviews.apache.org/r/44835/ I added a feature to > > support external properties file for complete structures like these. So you > > can create a json file at HDP/2.0.6/properties/stack_features.json and > > reference it in the config property xml file. I will be committing those > > changes soon. > > Juanjo Marron wrote: > I saw the example of stack_tools.json in > https://reviews.apache.org/r/44835/. > It would work even if the stack_fetaures.json file is defined out of the > stack, under HDP/configuration/stack_fetautes for example? New approach on https://reviews.apache.org/r/45141 uses the new feature to support external properties file - Juanjo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44933/#review124285 ----------------------------------------------------------- On March 22, 2016, 1:03 a.m., Juanjo Marron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44933/ > ----------------------------------------------------------- > > (Updated March 22, 2016, 1:03 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. > 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 > configuration/cluster-env.xml for each stack version. > 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. > 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. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/stack_fetaures.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 > d18e4d7 > > 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 > 3fb82e9 > > ambari-server/src/main/resources/stacks/HDP/2.2/configuration/cluster-env.xml > f1fa4de > > ambari-server/src/main/resources/stacks/HDP/2.3/configuration/cluster-env.xml > PRE-CREATION > > Diff: https://reviews.apache.org/r/44933/diff/ > > > Testing > ------- > > Zookeeper fresh installation has been tested using HDP stack 2.3 and > AMBARI-13364 branch code. > > > Thanks, > > Juanjo Marron > >
