Bug 1686433

Summary: proxy timeout loading list of services post update
Product: Red Hat CloudForms Management Engine Reporter: Felix Dewaleyne <fdewaley>
Component: ApplianceAssignee: Keenan Brock <kbrock>
Status: CLOSED CURRENTRELEASE QA Contact: Jaroslav Henner <jhenner>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.10.0CC: abellott, dmetzger, fdewaley, jprause, jrafanie, kbrock, mshriver, nlamuro, obarenbo, simaishi, smallamp
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.11.0   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.11.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1688937 (view as bug list) Environment:
Last Closed: 2019-12-13 14:54:24 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1688937    

Description Felix Dewaleyne 2019-03-07 13:04:46 UTC
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.

Comment 24 Keenan Brock 2019-03-13 21:32:58 UTC
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

Comment 25 CFME Bot 2019-03-14 17:42:28 UTC
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(-)

Comment 27 CFME Bot 2019-03-15 12:03:37 UTC
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(-)

Comment 28 Satoe Imaishi 2019-03-22 12:47:18 UTC
https://github.com/ManageIQ/manageiq/pull/18543

Comment 30 Jaroslav Henner 2019-08-19 09:01:22 UTC
I already started writing a test for this:
https://github.com/ManageIQ/integration_tests/pull/8698