Bug 1593798

Summary: Lifecycle VM Provision and Publish VM to Template Unusable/Slow
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: PerformanceAssignee: Nick LaMuro <nlamuro>
Status: CLOSED ERRATA QA Contact: Himanshu Roy <hroy>
Severity: high Docs Contact:
Priority: high    
Version: 5.8.0CC: cpelland, dluong, dmetzger, fwissing, jocarter, jrafanie, obarenbo, rovalent, simaishi, smallamp
Target Milestone: GAKeywords: ZStream
Target Release: 5.8.5   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.8.5.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1569626 Environment:
Last Closed: 2018-09-26 18:36:30 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On: 1569626    
Bug Blocks: 1572700    

Comment 2 CFME Bot 2018-06-21 18:41:40 UTC
New commits detected on ManageIQ/manageiq/fine:

https://github.com/ManageIQ/manageiq/commit/ef6b3f9c75781c49d77b4d71b36d57c6ed8a8d6a
commit ef6b3f9c75781c49d77b4d71b36d57c6ed8a8d6a
Author:     Brandon Dunne <brandondunne>
AuthorDate: Fri Apr 27 18:28:23 2018 -0400
Commit:     Brandon Dunne <brandondunne>
CommitDate: Fri Apr 27 18:28:23 2018 -0400

    Merge pull request #17355 from NickLaMuro/memoize_log_prefix_calls_in_miq_request_workflow

    Memoize _log.prefix calls
    (cherry picked from commit c2e01c9bf8426239d1f691d93fa21ef78997304f)

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

 app/models/miq_request_workflow.rb | 15 +-
 1 file changed, 10 insertions(+), 5 deletions(-)


https://github.com/ManageIQ/manageiq/commit/a6dc75ee3d0eb686d8c3dab9c37b8a4a4865a6d9
commit a6dc75ee3d0eb686d8c3dab9c37b8a4a4865a6d9
Author:     Keenan Brock <keenan>
AuthorDate: Fri May  4 10:40:36 2018 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Fri May  4 10:40:36 2018 -0400

    Merge pull request #17325 from NickLaMuro/flatten_arranged_rels_optimization

    Remove N+1 obj creation in flatten_arranged_rels
    (cherry picked from commit 32fadeb91c0e63e34efa4c9321ec52a6bdb176ae)

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

 app/models/relationship.rb | 11 +-
 1 file changed, 8 insertions(+), 3 deletions(-)


https://github.com/ManageIQ/manageiq/commit/3f5401bf13b776226e243f4eba3b336c9f61866a
commit 3f5401bf13b776226e243f4eba3b336c9f61866a
Author:     Keenan Brock <keenan>
AuthorDate: Fri May 18 12:08:20 2018 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Fri May 18 12:08:20 2018 -0400

    Merge pull request #17354 from NickLaMuro/sql_improvements_for_miq_request_workflow

    Prevents N+1 SQL queries in miq_request_workflow.rb
    (cherry picked from commit 65631b83ab169071c9691b130da84112de477ef6)

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

 app/models/host.rb | 12 +-
 app/models/miq_provision_virt_workflow.rb | 4 +-
 app/models/miq_request_workflow.rb | 2 +-
 3 files changed, 14 insertions(+), 4 deletions(-)


https://github.com/ManageIQ/manageiq/commit/347a98c00148a05acfeabf480285d49cddb68f26
commit 347a98c00148a05acfeabf480285d49cddb68f26
Author:     Keenan Brock <keenan>
AuthorDate: Fri May 18 12:12:14 2018 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Fri May 18 12:12:14 2018 -0400

    Merge pull request #17422 from NickLaMuro/avoid_duplicate_get_ems_folders_for_datacenters

    Add uniq on datacenters in #host_to_folder
    (cherry picked from commit 9b33e0b3919266f70096b3ccacfab0657c1839ab)

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

 app/models/miq_request_workflow.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


https://github.com/ManageIQ/manageiq/commit/831c6435387ec9b5d69355523d3c80df31c0f218
commit 831c6435387ec9b5d69355523d3c80df31c0f218
Author:     Keenan Brock <keenan>
AuthorDate: Fri May 18 12:21:35 2018 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Fri May 18 12:21:35 2018 -0400

    Merge pull request #17357 from NickLaMuro/nested_hashes_for_miq_request_workflow

    Use nested hashes instead of string keys
    (cherry picked from commit 5f22e971f034528138b6be42181ee4ba90461fdb)

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

 app/models/miq_request_workflow.rb | 11 +-
 spec/models/miq_request_workflow_spec.rb | 6 +-
 2 files changed, 11 insertions(+), 6 deletions(-)


https://github.com/ManageIQ/manageiq/commit/361707d59833a4b0b388e4f880b6e7cc9f022c84
commit 361707d59833a4b0b388e4f880b6e7cc9f022c84
Author:     Greg McCullough <gmccullo>
AuthorDate: Tue May 22 15:45:25 2018 -0400
Commit:     Greg McCullough <gmccullo>
CommitDate: Tue May 22 15:45:25 2018 -0400

    Merge pull request #17402 from NickLaMuro/lighter_queries_in_miq_request_workflow

    Avoid duplicate host load in allowed_hosts_obj
    (cherry picked from commit d577df7a6d6f28c24264a65b30d7011db1d6d71a)

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

 app/models/miq_request_workflow.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


https://github.com/ManageIQ/manageiq/commit/ec917a83d9a8fc2f135fb40c575776391d68b6aa
commit ec917a83d9a8fc2f135fb40c575776391d68b6aa
Author:     Greg McCullough <gmccullo>
AuthorDate: Tue May 22 17:27:56 2018 -0400
Commit:     Greg McCullough <gmccullo>
CommitDate: Tue May 22 17:27:56 2018 -0400

    Merge pull request #17358 from NickLaMuro/refactor_MiqRequestWorkflow_get_ems_folders

    Refactor get_ems_folders to create less strings
    (cherry picked from commit 7060b2df04b5364bf6164a0368b5ea7edab6209e)

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

 app/models/miq_request_workflow.rb | 12 +-
 1 file changed, 9 insertions(+), 3 deletions(-)


https://github.com/ManageIQ/manageiq/commit/f92ee1f15fd14b953aeed3ff8aaf73a29723000e
commit f92ee1f15fd14b953aeed3ff8aaf73a29723000e
Author:     Keenan Brock <keenan>
AuthorDate: Wed May 30 14:46:15 2018 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Wed May 30 14:46:15 2018 -0400

    Merge pull request #17360 from NickLaMuro/ancestry_patches

    Ancestry Patches
    (cherry picked from commit 095f4dce436fa9f9fdedca852b53c55c5f6cb6e0)

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

 app/models/orchestration_stack.rb | 2 +
 app/models/relationship.rb | 1 +
 app/models/service.rb | 1 +
 app/models/tenant.rb | 1 +
 lib/patches/ancestry_patch.rb | 70 +
 5 files changed, 75 insertions(+)


https://github.com/ManageIQ/manageiq/commit/a1062535ada05bcdb39c6194e30a64f647d32044
commit a1062535ada05bcdb39c6194e30a64f647d32044
Author:     Keenan Brock <keenan>
AuthorDate: Tue Jun  5 10:54:24 2018 -0400
Commit:     Keenan Brock <keenan>
CommitDate: Tue Jun  5 10:54:24 2018 -0400

    Merge pull request #17511 from NickLaMuro/ancestry_memoization_fixes

    Ancestry Patch updates/fixes
    (cherry picked from commit c6a6bedd2a09df590eb7fec6657bad08389b0296)

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

 app/models/tenant.rb | 2 +-
 lib/patches/ancestry_patch.rb | 10 +
 spec/models/tenant_spec.rb | 24 +
 3 files changed, 35 insertions(+), 1 deletion(-)

Comment 16 errata-xmlrpc 2018-09-26 18:36:30 UTC
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-2018:2745