> On July 8, 2016, 8:23 p.m., Jaimin Jetly wrote: > > Ship It!
Thanks Jaimin! - Shantanu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49721/#review141328 ----------------------------------------------------------- On July 6, 2016, 7:20 p.m., Shantanu Mundkur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49721/ > ----------------------------------------------------------- > > (Updated July 6, 2016, 7:20 p.m.) > > > Review request for Ambari and Jaimin Jetly. > > > Bugs: AMBARI-17580 > https://issues.apache.org/jira/browse/AMBARI-17580 > > > Repository: ambari > > > Description > ------- > > AMBARI-17580: Unable to add MASTER component on all nodes > > > While trying to add a MASTER component with cardinality ALL: > > org.apache.ambari.server.controller.spi.ResourceAlreadyExistsException: > Attempted to create a host_component which already exists: [clusterName=HDP, > hostName=node1.mydomain.com, componentName=SERVICE_MASTER] > > > Diffs > ----- > > ambari-web/app/controllers/wizard/step8_controller.js 3971cf5 > ambari-web/test/controllers/wizard/step8_test.js 74e042b > > Diff: https://reviews.apache.org/r/49721/diff/ > > > Testing > ------- > > 1) Tried to add a custom service with cardinality as ALL for a MASTER > component (that was previously the failing case). > > 2) Unit tests: > Under ambari-web: > mvn clean test > > Results: > 28946 tests complete (37 seconds) > 154 tests pending > > > Thanks, > > Shantanu Mundkur > >
