Re: Review Request 51795: ZEPPELIN user won't get created in ZEPPELIN group

2016-09-12 Thread Vitalyi Brodetskyi 

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


Ship it!




Ship It!

- Vitalyi Brodetskyi


On Вер. 12, 2016, 9:37 до полудня, Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51795/
> ---
> 
> (Updated Вер. 12, 2016, 9:37 до полудня)
> 
> 
> Review request for Ambari and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-18356
> https://issues.apache.org/jira/browse/AMBARI-18356
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Zeppelin user should also be created with group = ZEPPELIN. We should also 
> look at other
> services. Further we should look at how this user to group linkage can be
> config property driven instead of hardcoded logic in the before-ANY hook.
> 
>  service/blob/master/configuration/zeppelin-env.xml#L29-L41>  
>  server/src/main/resources/stacks/HDP/2.0.6/hooks/before-
> ANY/scripts/params.py#L209-L221>
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/params.py
>  5544085 
> 
> Diff: https://reviews.apache.org/r/51795/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>

Review Request 51795: ZEPPELIN user won't get created in ZEPPELIN group

2016-09-12 Thread Andrew Onischuk 

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

Review request for Ambari and Vitalyi Brodetskyi.


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


Repository: ambari


Description
---

Zeppelin user should also be created with group = ZEPPELIN. We should also look 
at other
services. Further we should look at how this user to group linkage can be
config property driven instead of hardcoded logic in the before-ANY hook.

  



Diffs
-

  
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-ANY/scripts/params.py
 5544085 

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


Testing
---

mvn clean test


Thanks,

Andrew Onischuk