Re: Review Request 51859: Enable Kerberos Wizard: Metrics collector fails to start on a cluster with Ranger installed

2016-09-13 Thread Sumit Mohanty

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


Ship it!




Ship It!

- Sumit Mohanty


On Sept. 13, 2016, 8:40 p.m., Jaimin Jetly wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51859/
> ---
> 
> (Updated Sept. 13, 2016, 8:40 p.m.)
> 
> 
> Review request for Ambari, Zhe (Joe) Wang, Richard Zang, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-18382
> https://issues.apache.org/jira/browse/AMBARI-18382
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> This happens due to incorrect config property update of Ambari metrics service
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 91a40dd 
> 
> Diff: https://reviews.apache.org/r/51859/diff/
> 
> 
> Testing
> ---
> 
> Verified that patch fixes the issue on a cluster.
> Verified that all ambari-web unit tests passes:
> 29244 tests complete (29 seconds)
> 154 tests pending
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>



Re: Review Request 51859: Enable Kerberos Wizard: Metrics collector fails to start on a cluster with Ranger installed

2016-09-13 Thread Yusaku Sako

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


Ship it!




Ship It!

- Yusaku Sako


On Sept. 13, 2016, 8:40 p.m., Jaimin Jetly wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51859/
> ---
> 
> (Updated Sept. 13, 2016, 8:40 p.m.)
> 
> 
> Review request for Ambari, Zhe (Joe) Wang, Richard Zang, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-18382
> https://issues.apache.org/jira/browse/AMBARI-18382
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> This happens due to incorrect config property update of Ambari metrics service
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 91a40dd 
> 
> Diff: https://reviews.apache.org/r/51859/diff/
> 
> 
> Testing
> ---
> 
> Verified that patch fixes the issue on a cluster.
> Verified that all ambari-web unit tests passes:
> 29244 tests complete (29 seconds)
> 154 tests pending
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>



Review Request 51859: Enable Kerberos Wizard: Metrics collector fails to start on a cluster with Ranger installed

2016-09-13 Thread Jaimin Jetly

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

Review request for Ambari, Zhe (Joe) Wang, Richard Zang, and Yusaku Sako.


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


Repository: ambari


Description
---

This happens due to incorrect config property update of Ambari metrics service


Diffs
-

  ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 91a40dd 

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


Testing
---

Verified that patch fixes the issue on a cluster.
Verified that all ambari-web unit tests passes:
29244 tests complete (29 seconds)
154 tests pending


Thanks,

Jaimin Jetly



Re: Review Request 51859: Enable Kerberos Wizard: Metrics collector fails to start on a cluster with Ranger installed

2016-09-13 Thread Zhe (Joe) Wang

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


Ship it!




Ship It!

- Zhe (Joe) Wang


On Sept. 13, 2016, 8:40 p.m., Jaimin Jetly wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51859/
> ---
> 
> (Updated Sept. 13, 2016, 8:40 p.m.)
> 
> 
> Review request for Ambari, Zhe (Joe) Wang, Richard Zang, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-18382
> https://issues.apache.org/jira/browse/AMBARI-18382
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> This happens due to incorrect config property update of Ambari metrics service
> 
> 
> Diffs
> -
> 
>   ambari-web/app/controllers/main/admin/kerberos/step4_controller.js 91a40dd 
> 
> Diff: https://reviews.apache.org/r/51859/diff/
> 
> 
> Testing
> ---
> 
> Verified that patch fixes the issue on a cluster.
> Verified that all ambari-web unit tests passes:
> 29244 tests complete (29 seconds)
> 154 tests pending
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>