Description of problem: The services explorer displays a tree view on the left hand side and the paginated list of services on the right. We did backport a fix that makes the left hand tree view responsive. We have not yet backported a fix to make the services list quick. The `report_data` query that loads the right hand side is taking minutes instead of a few seconds. You may get a timeout (meaning it took a really really long time) or you will notice that it took a few minutes. Version-Release number of selected component (if applicable): visible in 5.10.33 How reproducible: Every time if the customer has enough services defined. Probably close to 3,000 Steps to Reproduce: 1. load a high number of servies 2. visit the services/explorer with a list of services Actual results: It takes too long to load and often times out. Expected results: The page should load in less than 10 seconds Additional info: This was seen in https://bugzilla.redhat.com/show_bug.cgi?id=1686433 but the same basic bug rear's it's head every few months. The previous solution was to have the customer one off tweak the product/views/Service.yaml file. This BZ is here to change that decision and apply that fix to all customer's service views.
https://github.com/ManageIQ/manageiq-ui-classic/pull/5366
Assigning this to Jaroslav based on https://bugzilla.redhat.com/show_bug.cgi?id=1688937#c14
Works.
Wait. I got confused. This isn't the same issue as 1686433
https://github.com/ManageIQ/manageiq-ui-classic/pull/5366 was reverted in https://github.com/ManageIQ/manageiq-ui-classic/pull/5768, so putting back to ON_DEV
this was rolled back for 2 reasons: 1. the order was broken https://github.com/ManageIQ/manageiq/pull/18928 2. the distinct was forgotten https://github.com/ManageIQ/manageiq/pull/19025
Testig steps In the settings flag enable use_sql_view: :reporting: :use_sql_view: true The settings page (and vms page) should be much faster The number of records in the settings page should be the same it should still sort properly This is necessary for larger databases
New commit detected on ManageIQ/manageiq/ivanchuk: https://github.com/ManageIQ/manageiq/commit/6ff1fc0c25fd11e392ca4d94c10a8b0b40e992b7 commit 6ff1fc0c25fd11e392ca4d94c10a8b0b40e992b7 Author: Gregg Tanzillo <gtanzill> AuthorDate: Thu Aug 8 16:17:59 2019 -0400 Commit: Gregg Tanzillo <gtanzill> CommitDate: Thu Aug 8 16:17:59 2019 -0400 Merge pull request #19025 from kbrock/turbo_distinct Turbo distinct (cherry picked from commit ec3a2dcce6498c146c12d0384e9b07be0f33f7eb) https://bugzilla.redhat.com/show_bug.cgi?id=1718102 app/models/miq_report.rb | 2 +- config/settings.yml | 1 + lib/rbac/filterer.rb | 46 +- spec/lib/rbac/filterer_spec.rb | 98 + 4 files changed, 138 insertions(+), 9 deletions(-)
My automation test started failing on that the services tree doesn't contain items. It looks like it happens in Chrome, but not in Firefox.
Created attachment 1606452 [details] no svc tree
Turning use sql view on for 5.9 and 5.10 will fix the right hand side: https://github.com/ManageIQ/manageiq-ui-classic/pull/6101 This was merged Mon 2019-08-26 I'm unsure when the commit to fix the left hand nav bar was merged. But it was improved for performance.
The right hand side should be all set and was merged/backported to I/H H: https://github.com/ManageIQ/manageiq-ui-classic/pull/5785 I?: https://github.com/ManageIQ/manageiq-ui-classic/pull/5686 (Merged to master July 10)
sorry, I mistyped comment 16. It should read, the left hand side should be all set. It was resolved in https://bugzilla.redhat.com/show_bug.cgi?id=1727443
New commit detected on ManageIQ/manageiq-ui-classic/ivanchuk: https://github.com/ManageIQ/manageiq-ui-classic/commit/08d05c975c8e0596e254c2a627d801d9ce6876b9 commit 08d05c975c8e0596e254c2a627d801d9ce6876b9 Author: Milan Zázrivec <mzazrivec> AuthorDate: Tue Aug 27 04:10:12 2019 -0400 Commit: Milan Zázrivec <mzazrivec> CommitDate: Tue Aug 27 04:10:12 2019 -0400 Merge pull request #6101 from kbrock/turbo_services Turbo mode for services (cherry picked from commit 38a59e54dfd03f0934b776324226e07c8cd1193e) https://bugzilla.redhat.com/show_bug.cgi?id=1718102 product/views/Service.yaml | 3 + 1 file changed, 3 insertions(+)
Verified in Version 5.11.0.23.20190904213640_d113674