Description of problem: customer with 6602 services is experiencing timeouts displaying the list of services since their update to 4.7 - in 4.6 the list was displaying. Version-Release number of selected component (if applicable): 5.10.0.33 How reproducible: customer environment Steps to Reproduce: 1.use 4.6 with a high number of services 2.update to 5.10.0.33 3.load the list of services (service/explorer) Actual results: proxy timeout with apache taking over 320 seconds loading the page Expected results: it should take 5 seconds to display the page (behaviour in 4.6) Additional info: multiple appliance setup with dedicated webui appliance with 10 workers.
https://github.com/ManageIQ/manageiq-ui-classic/pull/5328
https://github.com/ManageIQ/manageiq-ui-classic/pull/5335
FYI, we did fix the double angular library issue, but since it looks like more of an annoyance and less of an issue, we only applied to master. If the client feels it is causing them performance problems, we can look into back porting it. https://github.com/ManageIQ/manageiq-ui-classic/pull/5330
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/dd426a631d2638fe5a5bb83d04a6247b7a7802a9 commit dd426a631d2638fe5a5bb83d04a6247b7a7802a9 Author: Dávid Halász <dhalasz> AuthorDate: Wed Mar 13 07:08:43 2019 -0400 Commit: Dávid Halász <dhalasz> CommitDate: Wed Mar 13 07:08:43 2019 -0400 Hide services from the Services tree as its redundant and slow Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1686433 app/presenters/tree_builder_services.rb | 16 +- spec/presenters/tree_builder_services_spec.rb | 15 - 2 files changed, 3 insertions(+), 28 deletions(-)
New commit detected on ManageIQ/manageiq-ui-classic/hammer: https://github.com/ManageIQ/manageiq-ui-classic/commit/1b32bca06c24b2e7cfe56cde92a2001c73b252ab commit 1b32bca06c24b2e7cfe56cde92a2001c73b252ab Author: Martin Hradil <mhradil> AuthorDate: Thu Mar 14 15:59:00 2019 -0400 Commit: Martin Hradil <mhradil> CommitDate: Thu Mar 14 15:59:00 2019 -0400 TreeBuilderServices - move allow_reselect to `set_locals_for_render` before #5290, `allow_reselect` needs to live in `set_locals_for_render`, not `tree_init_options` This fixes the hammer backport of https://github.com/ManageIQ/manageiq-ui-classic/pull/5335 to do just that. https://bugzilla.redhat.com/show_bug.cgi?id=1686433 app/presenters/tree_builder_services.rb | 3 +- 1 file changed, 1 insertion(+), 2 deletions(-)
https://github.com/ManageIQ/manageiq/pull/18543
I already started writing a test for this: https://github.com/ManageIQ/integration_tests/pull/8698