Re: Review Request 52497: Enable Add/Remove JournalNode on NNHA Wizard Step 2

2016-10-03 Thread Zhe (Joe) Wang

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


Ship it!




Ship It!

- Zhe (Joe) Wang


On Oct. 3, 2016, 11:13 p.m., Richard Zang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52497/
> ---
> 
> (Updated Oct. 3, 2016, 11:13 p.m.)
> 
> 
> Review request for Ambari, Jaimin Jetly, Zhe (Joe) Wang, Xi Wang, and Yusaku 
> Sako.
> 
> 
> Bugs: AMBARI-18519
> https://issues.apache.org/jira/browse/AMBARI-18519
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Enable Add/Remove JournalNode on NNHA Wizard Step 2
> 
> 
> Diffs
> -
> 
>   
> ambari-web/app/controllers/main/admin/highAvailability/nameNode/step2_controller.js
>  b8d7978 
>   ambari-web/app/templates/main/admin/highAvailability/nameNode/step3.hbs 
> 27653d3 
> 
> Diff: https://reviews.apache.org/r/52497/diff/
> 
> 
> Testing
> ---
> 
> Manually tested on live cluster.
> All unit tests passed.
> 
>   30366 tests complete (30 seconds)
>   151 tests pending
> 
> 
> Thanks,
> 
> Richard Zang
> 
>



Review Request 52497: Enable Add/Remove JournalNode on NNHA Wizard Step 2

2016-10-03 Thread Richard Zang

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

Review request for Ambari, Jaimin Jetly, Zhe (Joe) Wang, Xi Wang, and Yusaku 
Sako.


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


Repository: ambari


Description
---

Enable Add/Remove JournalNode on NNHA Wizard Step 2


Diffs
-

  
ambari-web/app/controllers/main/admin/highAvailability/nameNode/step2_controller.js
 b8d7978 
  ambari-web/app/templates/main/admin/highAvailability/nameNode/step3.hbs 
27653d3 

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


Testing
---

Manually tested on live cluster.
All unit tests passed.

  30366 tests complete (30 seconds)
  151 tests pending


Thanks,

Richard Zang