Bug 1727443
| Summary: | Building the services tree takes too long when having 1000+ services | ||
|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Dávid Halász <dhalasz> |
| Component: | UI - OPS | Assignee: | Dávid Halász <dhalasz> |
| Status: | CLOSED ERRATA | QA Contact: | Niyaz Akhtar Ansari <nansari> |
| Severity: | medium | Docs Contact: | Red Hat CloudForms Documentation <cloudforms-docs> |
| Priority: | medium | ||
| Version: | 5.10.5 | CC: | bmidwood, hkataria, jhenner, lavenel, mpovolny, mshriver, obarenbo, simaishi |
| Target Milestone: | GA | Keywords: | ZStream |
| Target Release: | 5.10.8 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | 5.10.8.0 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2019-08-13 08:35:35 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: | |||
|
Description
Dávid Halász
2019-07-06 11:21:48 UTC
David, for hammer branch, do we also need the followings? https://github.com/ManageIQ/manageiq-ui-classic/pull/5729 https://github.com/ManageIQ/manageiq-ui-classic/pull/5737 Those PRs go with bug #1703556. We had a separate PR for hammer branch (https://github.com/ManageIQ/manageiq-ui-classic/pull/5530) for that BZ, but the 2 PRs are hammer/yes. Yes New commits detected on ManageIQ/manageiq-ui-classic/hammer: https://github.com/ManageIQ/manageiq-ui-classic/commit/99c27e15cb74c66888bf513f34e4bd7c55c9a4e3 commit 99c27e15cb74c66888bf513f34e4bd7c55c9a4e3 Author: Martin Hradil <mhradil> AuthorDate: Fri Jun 21 07:25:56 2019 -0400 Commit: Martin Hradil <mhradil> CommitDate: Fri Jun 21 07:25:56 2019 -0400 Merge pull request #5729 from skateman/explorer-tree-init-object Do not let angular parse the treeview data as it can be 🐢 HUGE 🐌 (cherry picked from commit 6d088fe4db708aad3506d0b097a64f2b3c041ef1) https://bugzilla.redhat.com/show_bug.cgi?id=1727443 app/assets/javascripts/controllers/tree_view_controller.js | 5 +- app/assets/javascripts/miq_global.js | 1 + app/views/layouts/listnav/_explorer.html.haml | 4 +- app/views/layouts/listnav/_generic_object_definition_show_list_with_treeview.html.haml | 4 +- 4 files changed, 10 insertions(+), 4 deletions(-) https://github.com/ManageIQ/manageiq-ui-classic/commit/1d0e088f004ff498808692ac47cb069b0a438853 commit 1d0e088f004ff498808692ac47cb069b0a438853 Author: Milan Zázrivec <mzazrivec> AuthorDate: Mon Jun 24 02:16:43 2019 -0400 Commit: Milan Zázrivec <mzazrivec> CommitDate: Mon Jun 24 02:16:43 2019 -0400 Merge pull request #5737 from skateman/tree-data-delete Fix temporary tree data deletion in angular explorer trees (cherry picked from commit f9a1581ec70962ec1cc8383a66a472d6abb41b61) https://bugzilla.redhat.com/show_bug.cgi?id=1727443 app/assets/javascripts/controllers/tree_view_controller.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) New commit detected on ManageIQ/manageiq-ui-classic/hammer: https://github.com/ManageIQ/manageiq-ui-classic/commit/32ff2f351a1c42512be233c1c3a51fba57c951a6 commit 32ff2f351a1c42512be233c1c3a51fba57c951a6 Author: Dávid Halász <dhalasz> AuthorDate: Mon Jun 24 06:48:22 2019 -0400 Commit: Dávid Halász <dhalasz> CommitDate: Mon Jun 24 06:48:22 2019 -0400 Improve the performance the lazy-loading of services tree by caching (cherry picked from commit eb8e1f40128c6463f9e2580585920203a03bc315) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1727443 app/presenters/tree_builder_services.rb | 41 +- 1 file changed, 32 insertions(+), 9 deletions(-) Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2019:2466 (In reply to Niyaz Akhtar Ansari from comment #11) > Thanks David > > Now Processing the request is faster > Verified in Version 5.10.8.0.20190729222758_e725e22 Hi Niyaz. I already started writing a test for Bug 1686433 - proxy timeout loading list of services post update : https://github.com/ManageIQ/integration_tests/pull/8698 which I think is related |