You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Alejandro Fernandez <af...@hortonworks.com> on 2016/11/28 19:57:40 UTC

Re: Review Request 53980: AMBARI-18945 Allowing user to edit service name while enabling ranger plugin

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




ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/configuration/ranger-kms-security.xml (line 24)
<https://reviews.apache.org/r/53980/#comment227550>

    Existing clusters will change to ${cluster_name}_kms during Ambari Upgrade.
    
    New clusters will use Stack Advisor to get the same name.
    
    After this point, if the user wishes to change the cluster name then the value of this config will remain the same.
    
    Is this the intended behavior? If so, mark this issue as fixed.


- Alejandro Fernandez


On Nov. 22, 2016, 11:53 a.m., Mugdha Varadkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53980/
> -----------------------------------------------------------
> 
> (Updated Nov. 22, 2016, 11:53 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan Hurley, and Velmurugan Periasamy.
> 
> 
> Bugs: AMBARI-18945
>     https://issues.apache.org/jira/browse/AMBARI-18945
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Need to add a provision for creating default repository with static name or user provided name.
> 
> This will help user to provided service name under Ranger supported plugins config tab and can take dump of latest snapshot of all policies and use it while recreating another/multiple clusters with same service name.
> 
> 
> Diffs
> -----
> 
>   ambari-common/src/main/python/resource_management/libraries/functions/constants.py 3431495 
>   ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java bc3c120 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py 32e1510 
>   ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py 1f32c41 
>   ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py 64f67b9 
>   ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py bba0f3b 
>   ambari-server/src/main/resources/common-services/KAFKA/0.8.1/package/scripts/params.py 0cb88fe 
>   ambari-server/src/main/resources/common-services/KAFKA/0.9.0/configuration/ranger-kafka-security.xml 91061d1 
>   ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params_linux.py 4cefbda 
>   ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/configuration/ranger-kms-security.xml 95e653c 
>   ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/params.py c20bdde 
>   ambari-server/src/main/resources/common-services/STORM/0.10.0/configuration/ranger-storm-security.xml 983702f 
>   ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py e753e98 
>   ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py 4d42861 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json dd87b72 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/configuration/ranger-hbase-security.xml 1deb9e5 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/configuration/ranger-hdfs-security.xml cfd8a4f 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HIVE/configuration/ranger-hive-security.xml ce4074a 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/KNOX/configuration/ranger-knox-security.xml 9bd1079 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/ranger-yarn-security.xml 5f69962 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py 775dbab 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/ranger-atlas-security.xml ea0a026 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py f9a3a9a 
>   ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java 4135919 
>   ambari-server/src/test/python/stacks/2.0.6/configs/nn_eu.json a6ed6ce 
>   ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py 39cbbce 
>   ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py 7124307 
>   ambari-server/src/test/python/stacks/2.5/configs/ranger-kms-default.json 05cb78a 
>   ambari-server/src/test/python/stacks/2.5/configs/ranger-kms-secured.json 7054e8f 
> 
> Diff: https://reviews.apache.org/r/53980/diff/
> 
> 
> Testing
> -------
> 
> mvn -DskipPythonTests -Dtest=UpgradeCatalog250Test test
> 
> -------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=512m; support was removed in 8.0
> Running org.apache.ambari.server.upgrade.UpgradeCatalog250Test
> Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.548 sec - in org.apache.ambari.server.upgrade.UpgradeCatalog250Test
> 
> Results :
> 
> Tests run: 13, Failures: 0, Errors: 0, Skipped: 0
> 
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 30.347 s
> [INFO] Finished at: 2016-11-22T11:51:23+00:00
> [INFO] Final Memory: 63M/1107M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Mugdha Varadkar
> 
>


Re: Review Request 53980: AMBARI-18945 Allowing user to edit service name while enabling ranger plugin

Posted by Mugdha Varadkar <mu...@freestoneinfotech.com>.

> On Nov. 28, 2016, 7:57 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/configuration/ranger-kms-security.xml, line 24
> > <https://reviews.apache.org/r/53980/diff/1/?file=1568491#file1568491line24>
> >
> >     Existing clusters will change to ${cluster_name}_kms during Ambari Upgrade.
> >     
> >     New clusters will use Stack Advisor to get the same name.
> >     
> >     After this point, if the user wishes to change the cluster name then the value of this config will remain the same.
> >     
> >     Is this the intended behavior? If so, mark this issue as fixed.

Hi Alejandro,
Yes this is what it is planned.


- Mugdha


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


On Nov. 22, 2016, 11:53 a.m., Mugdha Varadkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53980/
> -----------------------------------------------------------
> 
> (Updated Nov. 22, 2016, 11:53 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan Hurley, and Velmurugan Periasamy.
> 
> 
> Bugs: AMBARI-18945
>     https://issues.apache.org/jira/browse/AMBARI-18945
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Need to add a provision for creating default repository with static name or user provided name.
> 
> This will help user to provided service name under Ranger supported plugins config tab and can take dump of latest snapshot of all policies and use it while recreating another/multiple clusters with same service name.
> 
> 
> Diffs
> -----
> 
>   ambari-common/src/main/python/resource_management/libraries/functions/constants.py 3431495 
>   ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog250.java bc3c120 
>   ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py 32e1510 
>   ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py 1f32c41 
>   ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py 64f67b9 
>   ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py bba0f3b 
>   ambari-server/src/main/resources/common-services/KAFKA/0.8.1/package/scripts/params.py 0cb88fe 
>   ambari-server/src/main/resources/common-services/KAFKA/0.9.0/configuration/ranger-kafka-security.xml 91061d1 
>   ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/params_linux.py 4cefbda 
>   ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/configuration/ranger-kms-security.xml 95e653c 
>   ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/params.py c20bdde 
>   ambari-server/src/main/resources/common-services/STORM/0.10.0/configuration/ranger-storm-security.xml 983702f 
>   ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/params_linux.py e753e98 
>   ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py 4d42861 
>   ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json dd87b72 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/configuration/ranger-hbase-security.xml 1deb9e5 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/configuration/ranger-hdfs-security.xml cfd8a4f 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/HIVE/configuration/ranger-hive-security.xml ce4074a 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/KNOX/configuration/ranger-knox-security.xml 9bd1079 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/ranger-yarn-security.xml 5f69962 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py 775dbab 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/ranger-atlas-security.xml ea0a026 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py f9a3a9a 
>   ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java 4135919 
>   ambari-server/src/test/python/stacks/2.0.6/configs/nn_eu.json a6ed6ce 
>   ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py 39cbbce 
>   ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py 7124307 
>   ambari-server/src/test/python/stacks/2.5/configs/ranger-kms-default.json 05cb78a 
>   ambari-server/src/test/python/stacks/2.5/configs/ranger-kms-secured.json 7054e8f 
> 
> Diff: https://reviews.apache.org/r/53980/diff/
> 
> 
> Testing
> -------
> 
> mvn -DskipPythonTests -Dtest=UpgradeCatalog250Test test
> 
> -------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=512m; support was removed in 8.0
> Running org.apache.ambari.server.upgrade.UpgradeCatalog250Test
> Tests run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.548 sec - in org.apache.ambari.server.upgrade.UpgradeCatalog250Test
> 
> Results :
> 
> Tests run: 13, Failures: 0, Errors: 0, Skipped: 0
> 
> 
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 30.347 s
> [INFO] Finished at: 2016-11-22T11:51:23+00:00
> [INFO] Final Memory: 63M/1107M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Mugdha Varadkar
> 
>