> On Sept. 12, 2017, 5:09 p.m., Ayub Pathan wrote: > > I dont see UTs added for BlueprintConfigurationProcessor.java. But the > > patch attached to the jira has the tests changes as well. Please check.
Thanks for pointing it out. I uploaded the wrong patch. - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62182/#review185192 ----------------------------------------------------------- On Sept. 12, 2017, 5:18 p.m., Sebastian Toader wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62182/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2017, 5:18 p.m.) > > > Review request for Ambari, Attila Magyar, Ayub Pathan, Robert Levas, and > Sandor Magyari. > > > Bugs: AMBARI-21902 > https://issues.apache.org/jira/browse/AMBARI-21902 > > > Repository: ambari > > > Description > ------- > > Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead > it should set to zookeeper quorum. The property updater in the blueprint > processor responsible for resolving this property needed to be reconfigured > to set this property to zookeeper server host name. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java > 3538945 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java > 734dd7e > > > Diff: https://reviews.apache.org/r/62182/diff/2/ > > > Testing > ------- > > Manual testing cluster creation using blueprints. > Testing Blueprint export. > > Unit tests: > [INFO] Results: > [INFO] > [WARNING] Tests run: 4870, Failures: 0, Errors: 0, Skipped: 37 > > > Thanks, > > Sebastian Toader > >