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

Ship it!


Ship It!

- Yusaku Sako


On July 15, 2015, 7:02 p.m., Jaimin Jetly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36521/
> -----------------------------------------------------------
> 
> (Updated July 15, 2015, 7:02 p.m.)
> 
> 
> Review request for Ambari, Srimanth Gunturi and Yusaku Sako.
> 
> 
> Bugs: AMBARI-12424
>     https://issues.apache.org/jira/browse/AMBARI-12424
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Steps:
> # Setup secure cluster with Storm.
> # Try to add Nimbus to Storm.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/main/host/details.js 709a92d 
> 
> Diff: https://reviews.apache.org/r/36521/diff/
> 
> 
> Testing
> -------
> 
> Tested the fix on a live cluster to resolve the issue. 
> I was able to add a nimbus host component and subsequent start of nimbus host 
> component happened successfully after applying the fix on the cluster failing 
> with the error previously.
> 
> ambari-web unit tests passess successfully with the patch:
> 
>   6662 tests complete (11 seconds)
>   90 tests pending
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>

Reply via email to