You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Jaimin Jetly <ja...@hortonworks.com> on 2015/06/29 05:45:26 UTC
Review Request 35996: Attempting to add service atlas does not show
identities in kerberized cluster
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35996/
-----------------------------------------------------------
Review request for Ambari, Richard Zang, Srimanth Gunturi, and Yusaku Sako.
Bugs: AMBARI-12197
https://issues.apache.org/jira/browse/AMBARI-12197
Repository: ambari
Description
-------
While adding a new service in kerberized cluster, we are using kerberos descriptor persisted at cluster resource endpoint. This kerberos descriptor has kerberos definition for all services present in the stack definition when kerberos wizard was executed.
When a kerberos supporting uninstalled service and non-existing in a stack of already kerberized cluster gets added later in the stack via HDP upgrade, Add service wizard triggered subsequently after upgrade to install that service will not show kerberos identities or configuration of that service for any user customizations.
As part of this patch, new services properties are fetched from stack descriptor instead of cluster descriptor. Also all the kerberos realted configuration (identities and properties) of a service in the stack descriptor that are not present in the cluster descriptor are shown for customization and applied to the cluster.
Diffs
-----
ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 6746f0a
ambari-web/app/controllers/main/service/add_controller.js a33ccaf
ambari-web/app/mixins/wizard/addSecurityConfigs.js 7e436ae
Diff: https://reviews.apache.org/r/35996/diff/
Testing
-------
Tested e2e.
Verified that all unit test for ambari-web passess.
Thanks,
Jaimin Jetly
Re: Review Request 35996: Attempting to add service atlas does not
show identities in kerberized cluster
Posted by Yusaku Sako <yu...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35996/#review89762
-----------------------------------------------------------
Ship it!
Ship It!
- Yusaku Sako
On June 29, 2015, 3:45 a.m., Jaimin Jetly wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35996/
> -----------------------------------------------------------
>
> (Updated June 29, 2015, 3:45 a.m.)
>
>
> Review request for Ambari, Richard Zang, Srimanth Gunturi, and Yusaku Sako.
>
>
> Bugs: AMBARI-12197
> https://issues.apache.org/jira/browse/AMBARI-12197
>
>
> Repository: ambari
>
>
> Description
> -------
>
> While adding a new service in kerberized cluster, we are using kerberos descriptor persisted at cluster resource endpoint. This kerberos descriptor has kerberos definition for all services present in the stack definition when kerberos wizard was executed.
>
> When a kerberos supporting uninstalled service and non-existing in a stack of already kerberized cluster gets added later in the stack via HDP upgrade, Add service wizard triggered subsequently after upgrade to install that service will not show kerberos identities or configuration of that service for any user customizations.
>
> As part of this patch, new services properties are fetched from stack descriptor instead of cluster descriptor. Also all the kerberos realted configuration (identities and properties) of a service in the stack descriptor that are not present in the cluster descriptor are shown for customization and applied to the cluster.
>
>
> Diffs
> -----
>
> ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 6746f0a
> ambari-web/app/controllers/main/service/add_controller.js a33ccaf
> ambari-web/app/mixins/wizard/addSecurityConfigs.js 7e436ae
>
> Diff: https://reviews.apache.org/r/35996/diff/
>
>
> Testing
> -------
>
> Tested e2e.
> Verified that all unit test for ambari-web passess.
>
>
> Thanks,
>
> Jaimin Jetly
>
>