ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitro Lisnichenko" <dlysniche...@hortonworks.com>
Subject Re: Review Request 40855: Exported blueprint does not contain "policymgr_external_url" property
Date Wed, 02 Dec 2015 11:07:39 GMT

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

Ship it!



ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py (line 314)
<https://reviews.apache.org/r/40855/#comment168081>

    maybe strip().isempty() ?


- Dmitro Lisnichenko


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message