Re: Review Request 49221: Smart configs for Atlas to setup authentication as either LDAP or AD

2016-06-24 Thread Alejandro Fernandez

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




ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml
 (line 262)


For now, set all props to required with value as " " til Atlas team can 
tell me which ones are required/optional.



ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 
(line 123)


Added smart config theme



ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py (line 
68)


This will no longer be needed once I know exactly which configs are 
required.


- Alejandro Fernandez


On June 25, 2016, 12:06 a.m., Alejandro Fernandez wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49221/
> ---
> 
> (Updated June 25, 2016, 12:06 a.m.)
> 
> 
> Review request for Ambari, Jaimin Jetly, Abhay Kulkarni, Robert Levas, Swapan 
> Shridhar, and Suma Shivaprasad.
> 
> 
> Bugs: AMBARI-17221
> https://issues.apache.org/jira/browse/AMBARI-17221
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> LDAP/AD authentication to Atlas was added in ATLAS-869. Ambari should support 
> Smart configs for Atlas to either use LDAP or AD.
> 
> Following configurations should be shown only when 
> "atlas.authentication.method.ldap.type" is set to "ldap":
> 
> * atlas.authentication.method.ldap.url=
> * atlas.authentication.method.ldap.userDNpattern=uid=
> * atlas.authentication.method.ldap.groupSearchBase=
> * atlas.authentication.method.ldap.groupSearchFilter=
> * atlas.authentication.method.ldap.groupRoleAttribute=cn
> * atlas.authentication.method.ldap.base.dn=
> * atlas.authentication.method.ldap.bind.dn=
> * atlas.authentication.method.ldap.bind.password=
> * atlas.authentication.method.ldap.referral=ignore
> * atlas.authentication.method.ldap.user.searchfilter=
> * atlas.authentication.method.ldap.default.role=ROLE_USER
> 
> Following configurations should be shown only when 
> "atlas.authentication.method.ldap.type" is set to "ad":
> 
> * atlas.authentication.method.ldap.ad.domain=
> * atlas.authentication.method.ldap.ad.url=
> * atlas.authentication.method.ldap.ad.base.dn=
> * atlas.authentication.method.ldap.ad.bind.dn=
> * atlas.authentication.method.ldap.ad.bind.password=
> * atlas.authentication.method.ldap.ad.referral=ignore
> * atlas.authentication.method.ldap.ad.user.searchfilter=(sAMAccountName={0})
> * atlas.authentication.method.ldap.ad.default.role=ROLE_USER
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml
>  25df1c5 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 
> 9f3eda9 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/themes/theme.json
>  PRE-CREATION 
>   
> ambari-server/src/main/resources/common-services/RANGER/0.5.0/configuration/ranger-admin-site.xml
>  5606ce6 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml
>  a5a63ba 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 614f0b1 
> 
> Diff: https://reviews.apache.org/r/49221/diff/
> 
> 
> Testing
> ---
> 
> Installed Atlas with HDP 2.5
> 
> Python unit tests passed.
> 
> ---
> Total run:1061
> Total errors:0
> Total failures:0
> OK
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>



Review Request 49221: Smart configs for Atlas to setup authentication as either LDAP or AD

2016-06-24 Thread Alejandro Fernandez

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

Review request for Ambari, Jaimin Jetly, Abhay Kulkarni, Robert Levas, Swapan 
Shridhar, and Suma Shivaprasad.


Bugs: AMBARI-17221
https://issues.apache.org/jira/browse/AMBARI-17221


Repository: ambari


Description
---

LDAP/AD authentication to Atlas was added in ATLAS-869. Ambari should support 
Smart configs for Atlas to either use LDAP or AD.

Following configurations should be shown only when 
"atlas.authentication.method.ldap.type" is set to "ldap":

* atlas.authentication.method.ldap.url=
* atlas.authentication.method.ldap.userDNpattern=uid=
* atlas.authentication.method.ldap.groupSearchBase=
* atlas.authentication.method.ldap.groupSearchFilter=
* atlas.authentication.method.ldap.groupRoleAttribute=cn
* atlas.authentication.method.ldap.base.dn=
* atlas.authentication.method.ldap.bind.dn=
* atlas.authentication.method.ldap.bind.password=
* atlas.authentication.method.ldap.referral=ignore
* atlas.authentication.method.ldap.user.searchfilter=
* atlas.authentication.method.ldap.default.role=ROLE_USER

Following configurations should be shown only when 
"atlas.authentication.method.ldap.type" is set to "ad":

* atlas.authentication.method.ldap.ad.domain=
* atlas.authentication.method.ldap.ad.url=
* atlas.authentication.method.ldap.ad.base.dn=
* atlas.authentication.method.ldap.ad.bind.dn=
* atlas.authentication.method.ldap.ad.bind.password=
* atlas.authentication.method.ldap.ad.referral=ignore
* atlas.authentication.method.ldap.ad.user.searchfilter=(sAMAccountName={0})
* atlas.authentication.method.ldap.ad.default.role=ROLE_USER


Diffs
-

  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/application-properties.xml
 25df1c5 
  ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/metainfo.xml 
9f3eda9 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/themes/theme.json
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/RANGER/0.5.0/configuration/ranger-admin-site.xml
 5606ce6 
  
ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml
 a5a63ba 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
614f0b1 

Diff: https://reviews.apache.org/r/49221/diff/


Testing
---

Installed Atlas with HDP 2.5

Python unit tests passed.

---
Total run:1061
Total errors:0
Total failures:0
OK


Thanks,

Alejandro Fernandez