> On March 9, 2016, 2:13 p.m., Alexandr Antonenko wrote:
> > Ship It!

Thanks Alexandr. Can you please help push the fix?


- Sangeeta


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


On March 8, 2016, 5:50 p.m., Sangeeta Ravindran wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44504/
> -----------------------------------------------------------
> 
> (Updated March 8, 2016, 5:50 p.m.)
> 
> 
> Review request for Ambari, Alexandr Antonenko and Jaimin Jetly.
> 
> 
> Bugs: AMBARI-15333
>     https://issues.apache.org/jira/browse/AMBARI-15333
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> When Host Check is run during cluster install, in Step 3 of the Installation 
> wizard, if you click on "Click here to see the warnings" and expand User 
> Issues, the message displayed incorrectly shows that a user exists on 
> multiple hosts (see attachment) instead of listing the service users that 
> exist on the host.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/wizard/step3_controller.js 19ddd20 
>   ambari-web/test/controllers/wizard/step3_test.js dbfb868 
> 
> Diff: https://reviews.apache.org/r/44504/diff/
> 
> 
> Testing
> -------
> 
> The issue seems to happen during parsing of the Host check warnings. In 
> parseHostCheckWarnings, the warningCategories.usersWarnings array elements 
> are accessed based on user.userName which is not a property of the 
> existingUsers objects. Fix is to use user.name instead.
> 
> Manual testing.
> mvn test
> 24565 tests complete (31 seconds)
> 145 tests pending
> 
> Updated test cases accordingly.
> 
> 
> Thanks,
> 
> Sangeeta Ravindran
> 
>

Reply via email to