You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Andrii Babiichuk <ab...@hortonworks.com> on 2017/05/17 18:15:52 UTC

Review Request 59347: Add service wizard unresponsive while adding HST via Ambari

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

Review request for Ambari and Aleksandr Kovalenko.


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


Repository: ambari


Description
-------

While adding HST, the 'add service wizard' stay stuck after clicking next button after choosing the target hosts. 

Exception in browser console logs -
Uncaught DOMException: Failed to execute 'setItem' on 'Storage': Setting the value of 'ambari' exceeded the quota.


Diffs
-----

  ambari-web/app/controllers/installer.js ec280fa 
  ambari-web/app/controllers/main/admin/highAvailability/journalNode/step1_controller.js b54986b 
  ambari-web/app/controllers/main/service/manage_config_groups_controller.js 45c840a 
  ambari-web/app/controllers/wizard.js a8a0249 
  ambari-web/app/controllers/wizard/step7/assign_master_controller.js 099931a 
  ambari-web/app/mixins/wizard/assign_master_components.js 3e2a09a 
  ambari-web/app/mixins/wizard/wizardHostsLoading.js 93dab02 
  ambari-web/app/routes/add_service_routes.js 89a4a36 
  ambari-web/app/routes/installer.js daefa48 
  ambari-web/app/utils/ajax/ajax.js 0b584d8 
  ambari-web/app/views/common/assign_master_components_view.js 892bcae 
  ambari-web/app/views/main/service/reassign_view.js 6885726 
  ambari-web/test/controllers/installer_test.js 94af88e 
  ambari-web/test/controllers/main/admin/highAvailability/journalNode/step1_controller_test.js 17f5ed2 
  ambari-web/test/controllers/main/service/add_controller_test.js a52c38f 
  ambari-web/test/controllers/wizard/step5_test.js 3c9048d 
  ambari-web/test/controllers/wizard_test.js 078f8ae 
  ambari-web/test/views/main/admin/highAvailability/resourceManager/wizard_view_test.js d30be42 
  ambari-web/test/views/main/service/reassign_view_test.js f6ae81e 


Diff: https://reviews.apache.org/r/59347/diff/1/


Testing
-------

20698 passing (24s)
  128 pending


Thanks,

Andrii Babiichuk


Re: Review Request 59347: Add service wizard unresponsive while adding HST via Ambari

Posted by Aleksandr Kovalenko <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59347/#review175296
-----------------------------------------------------------


Ship it!




Ship It!

- Aleksandr Kovalenko


On Май 17, 2017, 6:15 п.п., Andrii Babiichuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59347/
> -----------------------------------------------------------
> 
> (Updated Май 17, 2017, 6:15 п.п.)
> 
> 
> Review request for Ambari and Aleksandr Kovalenko.
> 
> 
> Bugs: AMBARI-21044
>     https://issues.apache.org/jira/browse/AMBARI-21044
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> While adding HST, the 'add service wizard' stay stuck after clicking next button after choosing the target hosts. 
> 
> Exception in browser console logs -
> Uncaught DOMException: Failed to execute 'setItem' on 'Storage': Setting the value of 'ambari' exceeded the quota.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/installer.js ec280fa 
>   ambari-web/app/controllers/main/admin/highAvailability/journalNode/step1_controller.js b54986b 
>   ambari-web/app/controllers/main/service/manage_config_groups_controller.js 45c840a 
>   ambari-web/app/controllers/wizard.js a8a0249 
>   ambari-web/app/controllers/wizard/step7/assign_master_controller.js 099931a 
>   ambari-web/app/mixins/wizard/assign_master_components.js 3e2a09a 
>   ambari-web/app/mixins/wizard/wizardHostsLoading.js 93dab02 
>   ambari-web/app/routes/add_service_routes.js 89a4a36 
>   ambari-web/app/routes/installer.js daefa48 
>   ambari-web/app/utils/ajax/ajax.js 0b584d8 
>   ambari-web/app/views/common/assign_master_components_view.js 892bcae 
>   ambari-web/app/views/main/service/reassign_view.js 6885726 
>   ambari-web/test/controllers/installer_test.js 94af88e 
>   ambari-web/test/controllers/main/admin/highAvailability/journalNode/step1_controller_test.js 17f5ed2 
>   ambari-web/test/controllers/main/service/add_controller_test.js a52c38f 
>   ambari-web/test/controllers/wizard/step5_test.js 3c9048d 
>   ambari-web/test/controllers/wizard_test.js 078f8ae 
>   ambari-web/test/views/main/admin/highAvailability/resourceManager/wizard_view_test.js d30be42 
>   ambari-web/test/views/main/service/reassign_view_test.js f6ae81e 
> 
> 
> Diff: https://reviews.apache.org/r/59347/diff/1/
> 
> 
> Testing
> -------
> 
> 20698 passing (24s)
>   128 pending
> 
> 
> Thanks,
> 
> Andrii Babiichuk
> 
>