You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Vitalyi Brodetskyi <vb...@hortonworks.com> on 2015/12/28 18:20:50 UTC

Review Request 41744: Certain config properties get reverted when adding new host or host component (secure cluster)

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

Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Robert Levas, and Sumit Mohanty.


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


Repository: ambari


Description
-------

1) install ambari(with oozie) and enable kerberos
2) change "oozie.authentication.type" from "kerberos" to any class handler
3) add host with oozie client or server. As the result, all configurations(security) for oozie will be updated. And "oozie.authentication.type" will become "kerberos".
the same bug will appear when you will add new oozie server to existing secure cluster with oozie service.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java 3dae467 
  ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java bdb208b 
  ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java a12c1af 
  ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterConfigurationRequest.java 6e8b8a3 
  ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java 29949a4 
  ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterConfigurationRequestTest.java df32684 

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


Testing
-------

mvn clean test


Thanks,

Vitalyi Brodetskyi


Re: Review Request 41744: Certain config properties get reverted when adding new host or host component (secure cluster)

Posted by Robert Levas <rl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41744/#review112038
-----------------------------------------------------------

Ship it!


Ship It!

- Robert Levas


On Dec. 28, 2015, 12:20 p.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41744/
> -----------------------------------------------------------
> 
> (Updated Dec. 28, 2015, 12:20 p.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Robert Levas, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-14506
>     https://issues.apache.org/jira/browse/AMBARI-14506
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1) install ambari(with oozie) and enable kerberos
> 2) change "oozie.authentication.type" from "kerberos" to any class handler
> 3) add host with oozie client or server. As the result, all configurations(security) for oozie will be updated. And "oozie.authentication.type" will become "kerberos".
> the same bug will appear when you will add new oozie server to existing secure cluster with oozie service.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java 3dae467 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java bdb208b 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java a12c1af 
>   ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterConfigurationRequest.java 6e8b8a3 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java 29949a4 
>   ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterConfigurationRequestTest.java df32684 
> 
> Diff: https://reviews.apache.org/r/41744/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>


Re: Review Request 41744: Certain config properties get reverted when adding new host or host component (secure cluster)

Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41744/#review112149
-----------------------------------------------------------

Ship it!


Ship It!

- Dmitro Lisnichenko


On Dec. 28, 2015, 7:20 p.m., Vitalyi Brodetskyi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41744/
> -----------------------------------------------------------
> 
> (Updated Dec. 28, 2015, 7:20 p.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Robert Levas, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-14506
>     https://issues.apache.org/jira/browse/AMBARI-14506
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1) install ambari(with oozie) and enable kerberos
> 2) change "oozie.authentication.type" from "kerberos" to any class handler
> 3) add host with oozie client or server. As the result, all configurations(security) for oozie will be updated. And "oozie.authentication.type" will become "kerberos".
> the same bug will appear when you will add new oozie server to existing secure cluster with oozie service.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java 3dae467 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java bdb208b 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java a12c1af 
>   ambari-server/src/main/java/org/apache/ambari/server/topology/ClusterConfigurationRequest.java 6e8b8a3 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java 29949a4 
>   ambari-server/src/test/java/org/apache/ambari/server/topology/ClusterConfigurationRequestTest.java df32684 
> 
> Diff: https://reviews.apache.org/r/41744/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Vitalyi Brodetskyi
> 
>