> On Dec. 2, 2015, 1:07 p.m., Dmitro Lisnichenko wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py, > > line 314 > > <https://reviews.apache.org/r/40855/diff/1/?file=1150098#file1150098line314> > > > > maybe strip().isempty() ?
Thinking a bit more, we anyway don't expect tab/newline here - Dmitro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40855/#review108629 ----------------------------------------------------------- On Dec. 2, 2015, 9:37 a.m., Sebastian Toader wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40855/ > ----------------------------------------------------------- > > (Updated Dec. 2, 2015, 9:37 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Robert Nettleton. > > > Bugs: AMBARI-14148 > https://issues.apache.org/jira/browse/AMBARI-14148 > > > Repository: ambari > > > Description > ------- > > In case of HA deployment the "policy_external_url" is not automatically > constructed from the host and port of Ranger Admin but rather is the load > balancer URL that is on front of the Ranger Admin instances which is provided > by user. > > This property has been added to the list of properties that can support HA > using a nameservice name otherwise this property is ignored by Blueprint > export when it's format differs from cluster_host:port > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java > 7f11bb9 > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py > d4d0e83 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java > 17c0157 > > Diff: https://reviews.apache.org/r/40855/diff/ > > > Testing > ------- > > Manual testing: > 1. Verified export with both cases when Ranger is deployed with UI and using > Blueprint > > Unit tests: > ---------------------------------------------------------------------- > Total run:804 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Sebastian Toader > >
