> 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 > >
