Bug 1703556

Summary: Unable to view service tree hierarchy
Product: Red Hat CloudForms Management Engine Reporter: Ryan Spagnola <rspagnol>
Component: AutomateAssignee: Dávid Halász <dhalasz>
Status: CLOSED CURRENTRELEASE QA Contact: Niyaz Akhtar Ansari <nansari>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.10.0CC: dhalasz, dmetzger, fdewaley, gmccullo, hkataria, jhenner, lavenel, mkanoor, nansari, obarenbo, simaishi, tfitzger
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.11.0   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.11.0.14 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1711981 (view as bug list) Environment:
Last Closed: 2019-12-13 15:00:42 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1704905, 1711981    

Description Ryan Spagnola 2019-04-26 17:03:24 UTC
Description of problem:
The tree for services was removed in order to improve performance, however, the customer's services are now no longer organized correctly.

Version-Release number of selected component (if applicable):
5.10

How reproducible:
Always

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
Opening this bug for tracking purposes based on the internal discussion.

Comment 3 Dave Johnson 2019-04-26 18:01:32 UTC
Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set the severity to Low.

Comment 8 Tina Fitzgerald 2019-05-03 13:45:01 UTC
Hi Harpreet/Loic,

I agree that a Hammer only PR makes sense while we work on a better solution.

Thanks,
Tina

Comment 15 CFME Bot 2019-06-21 11:26:29 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/02a4883b7359e2481b7a9b4820913163129390f3
commit 02a4883b7359e2481b7a9b4820913163129390f3
Author:     Dávid Halász <dhalasz>
AuthorDate: Thu Jun 20 06:00:34 2019 -0400
Commit:     Dávid Halász <dhalasz>
CommitDate: Thu Jun 20 06:00:34 2019 -0400

    Do not let angular parse the treeview data as it can be :turtle: HUGE :snail:

    https://bugzilla.redhat.com/show_bug.cgi?id=1703556

 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(-)

Comment 16 CFME Bot 2019-07-10 10:42:00 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/610649b57d7516494db6b770f1b476be59ab61fa
commit 610649b57d7516494db6b770f1b476be59ab61fa
Author:     Dávid Halász <dhalasz>
AuthorDate: Sat Jun  8 04:51:27 2019 -0400
Commit:     Dávid Halász <dhalasz>
CommitDate: Sat Jun  8 04:51:27 2019 -0400

    Revert "Hide services from the Services tree as its redundant and slow"

    This reverts commit dd426a631d2638fe5a5bb83d04a6247b7a7802a9.

    https://bugzilla.redhat.com/show_bug.cgi?id=1703556

 app/presenters/tree_builder_services.rb | 16 +-
 spec/presenters/tree_builder_services_spec.rb | 15 +
 2 files changed, 28 insertions(+), 3 deletions(-)

Comment 18 Niyaz Akhtar Ansari 2019-07-17 06:46:15 UTC
Verified in Version 5.11.0.14.20190710225033_cb17ff0

Comment 19 Jaroslav Henner 2019-08-19 09:03:20 UTC
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