Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

2016-12-06 Thread alice alice

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

(Updated 十二月 7, 2016, 12:53 a.m.)


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


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


Repository: ambari


Description (updated)
---

1. when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' 
and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config 
these.


Diffs
-

  
ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js
 36dcfb8 
  ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
  ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 

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


Testing
---

done it


Thanks,

alice alice



Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

2016-12-06 Thread alice alice


> On 十二月 6, 2016, 3:03 p.m., Jonathan Hurley wrote:
> > I normally don't review web client requests since I have little experience 
> > with that part of the codebase...

OK,thank you all the same for your swift respondence.


- alice


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


On 十二月 6, 2016, 2:31 a.m., alice alice wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51640/
> ---
> 
> (Updated 十二月 6, 2016, 2:31 a.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-18312
> https://issues.apache.org/jira/browse/AMBARI-18312
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> 1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' 
> and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config 
> these.
> 
> 
> Diffs
> -
> 
>   
> ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js
>  36dcfb8 
>   ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
>   ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 
> 
> Diff: https://reviews.apache.org/r/51640/diff/
> 
> 
> Testing
> ---
> 
> done it
> 
> 
> Thanks,
> 
> alice alice
> 
>



Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

2016-12-05 Thread alice alice

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

(Updated 十二月 6, 2016, 2:31 a.m.)


Review request for Ambari, Jonathan Hurley and Vitalyi Brodetskyi.


Changes
---

when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 
'yarn.resoucemanager.admin.address.rm2' are missing;Maybe we need to config 
these when starting rm ha.


Summary (updated)
-

After staring rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 
'yarn.resoucemanager.admin.address.rm2' are missing.


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


Repository: ambari


Description
---

1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' 
and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config 
these.


Diffs
-

  
ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js
 36dcfb8 
  ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
  ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 

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


Testing
---

done it


Thanks,

alice alice



Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing

2016-09-06 Thread alice alice

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

(Updated 九月 6, 2016, 8:01 a.m.)


Review request for Ambari, Andrew Onischuk and Vitalyi Brodetskyi.


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


Repository: ambari


Description
---

1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' 
and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config 
these.


Diffs
-

  
ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js
 36dcfb8 
  ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
  ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 

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


Testing
---

done it


Thanks,

alice alice



Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing

2016-09-05 Thread alice alice

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

Review request for Ambari, Dmytro Sen, Jaimin Jetly, Srimanth Gunturi, Sid 
Wagle, and Vitalyi Brodetskyi.


Repository: ambari


Description
---

1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' 
and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config 
these.


Diffs
-

  
ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js
 36dcfb8 
  ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
  ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 

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


Testing
---

done it


Thanks,

alice alice