> On March 4, 2015, 9:24 p.m., Robert Levas wrote:
> > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/kerberos.json,
> > lines 109-132
> > <https://reviews.apache.org/r/31744/diff/1/?file=884764#file884764line109>
> >
> > Why is the supervisor component removed? If it is not there, the
> > component will not be considered for Kerberization. At least, an empty
> > entry is needed.
> >
> > {
> > "name" : "SUPERVISOR"
> > }
Storm principal and keytab already exists as service level identity for storm
and is repeated over here for supervisors. So the intention was to remove the
supervisor identity. Thanks for clarifying the need for presence of
{
"name" : "SUPERVISOR"
}
I have added it back in 2nd revision of the patch.
- Jaimin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31744/#review75238
-----------------------------------------------------------
On March 4, 2015, 10:56 p.m., Jaimin Jetly wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31744/
> -----------------------------------------------------------
>
> (Updated March 4, 2015, 10:56 p.m.)
>
>
> Review request for Ambari, Robert Levas, Srimanth Gunturi, and Yusaku Sako.
>
>
> Bugs: AMBARI-9936
> https://issues.apache.org/jira/browse/AMBARI-9936
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Deploy cluster with Hbase and Storm
> Start Security wizard. Select Active Directory mode.
> Proceed to Identity configuration step.
> There I can configure smokeuser and hdfs headless principal names. But for
> Hbase there is no field for headless principal configuration.
> Same for Storm. There is no field for headless principal configuration.
>
>
> Diffs
> -----
>
>
> ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/kerberos.json
> aa4fa75
> ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 06208ee
>
> Diff: https://reviews.apache.org/r/31744/diff/
>
>
> Testing
> -------
>
> Tested the patch end-to-end by successfully going through kerberos wizard
> with customized values for HBase and Storm
>
> **ambari-web unit test result with the patch:**
>
> 5479 tests complete (9 seconds)
> 62 tests pending
>
>
> Thanks,
>
> Jaimin Jetly
>
>
