Review Request 54865: Ambari UI should use "hosts_with_failures" from "host_resolution_check" in host check

2016-12-19 Thread Andrii Tkach 

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

Review request for Ambari and Aleksandr Kovalenko.


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


Repository: ambari


Description
---

Earlier UI parsed "failures" property value to get hosts which were failed. But 
in perf clusters "failures" is very big (more than 300 MB), so we decided to 
add one more property which contains only hostnames which failed.


Diffs
-

  ambari-web/app/mixins/main/host/details/actions/check_host.js faf8ef1 
  ambari-web/app/utils/ajax/ajax.js 5831521 
  ambari-web/test/controllers/wizard/step3_test.js 1e1b3bd 

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


Testing
---

19725 passing (25s)
  153 pending


Thanks,

Andrii Tkach

Re: Review Request 54865: Ambari UI should use "hosts_with_failures" from "host_resolution_check" in host check

2016-12-19 Thread Aleksandr Kovalenko 

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


Ship it!




Ship It!

- Aleksandr Kovalenko


On Дек. 19, 2016, 5:02 п.п., Andrii Tkach wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54865/
> ---
> 
> (Updated Дек. 19, 2016, 5:02 п.п.)
> 
> 
> Review request for Ambari and Aleksandr Kovalenko.
> 
> 
> Bugs: AMBARI-19240
> https://issues.apache.org/jira/browse/AMBARI-19240
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Earlier UI parsed "failures" property value to get hosts which were failed. 
> But in perf clusters "failures" is very big (more than 300 MB), so we decided 
> to add one more property which contains only hostnames which failed.
> 
> 
> Diffs
> -
> 
>   ambari-web/app/mixins/main/host/details/actions/check_host.js faf8ef1 
>   ambari-web/app/utils/ajax/ajax.js 5831521 
>   ambari-web/test/controllers/wizard/step3_test.js 1e1b3bd 
> 
> Diff: https://reviews.apache.org/r/54865/diff/
> 
> 
> Testing
> ---
> 
> 19725 passing (25s)
>   153 pending
> 
> 
> Thanks,
> 
> Andrii Tkach
> 
>