Re: Review Request 45045: enable ssl for ambari does not update the tez view url in tez-site.xml

2016-03-21 Thread Vitalyi Brodetskyi

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


Ship it!




Ship It!

- Vitalyi Brodetskyi


On March 18, 2016, 7:10 p.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45045/
> ---
> 
> (Updated March 18, 2016, 7:10 p.m.)
> 
> 
> Review request for Ambari, Sumit Mohanty and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-15484
> https://issues.apache.org/jira/browse/AMBARI-15484
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> When one enables ssl for ambari we need to update the url set in tez.tez-ui
> .history-url.base. We should update that to the new https url or atleast add
> that to the enable ssl doc for ambari so the users know they have to update
> it.
> 
> 
> Diffs
> -
> 
>   ambari-server/src/main/python/ambari_server/setupHttps.py c9fe421 
>   ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_advisor.py 
> 5e23b8a 
>   ambari-server/src/test/python/TestAmbariServer.py a1b9cac 
>   ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py 
> 9b2da28 
> 
> Diff: https://reviews.apache.org/r/45045/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Review Request 45045: enable ssl for ambari does not update the tez view url in tez-site.xml

2016-03-19 Thread Andrew Onischuk

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

Review request for Ambari and Sumit Mohanty.


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


Repository: ambari


Description
---

When one enables ssl for ambari we need to update the url set in tez.tez-ui
.history-url.base. We should update that to the new https url or atleast add
that to the enable ssl doc for ambari so the users know they have to update
it.


Diffs
-

  ambari-server/src/main/python/ambari_server/setupHttps.py c9fe421 
  ambari-server/src/main/resources/stacks/HDP/2.2/services/stack_advisor.py 
5e23b8a 
  ambari-server/src/test/python/TestAmbariServer.py a1b9cac 
  ambari-server/src/test/python/stacks/2.2/common/test_stack_advisor.py 9b2da28 

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


Testing
---

mvn clean test


Thanks,

Andrew Onischuk