Bug 1412729 - Provider refresh duration runs so long that multiple ems_worker processes for the same provider become active concurrently
Summary: Provider refresh duration runs so long that multiple ems_worker processes for...
Keywords:
Status: ON_QA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Performance
Version: 5.4.0
Hardware: x86_64
OS: Linux
medium
high
Target Milestone: GA
: 5.7.2
Assignee: Ladislav Smola
QA Contact: Pradeep Kumar Surisetty
URL:
Whiteboard: ems_refresh:perf
Depends On: 1251154
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-01-12 16:11 UTC by Satoe Imaishi
Modified: 2021-04-22 03:15 UTC (History)
14 users (show)

Fixed In Version: 5.7.1.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1251154
Environment:
Last Closed:
Category: ---
Cloudforms Team: ---
Target Upstream Version:


Attachments (Terms of Use)

Comment 2 CFME Bot 2017-01-12 16:15:53 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/fa933b82be02f62b847e576441da93346baeff91

commit fa933b82be02f62b847e576441da93346baeff91
Author:     Adam Grare <agrare@redhat.com>
AuthorDate: Thu Nov 3 16:49:59 2016 -0400
Commit:     Satoe Imaishi <simaishi@redhat.com>
CommitDate: Thu Jan 12 11:12:38 2017 -0500

    Merge pull request #12360 from kbrock/refresh_geneology
    
    refresh geneology use hash O(n) instead of O(n^2)
    (cherry picked from commit 0a2da0a1856738e43319d2a65ae821a295b926fd)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1412729

 app/models/ems_refresh/save_inventory.rb | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

Comment 3 CFME Bot 2017-01-12 16:16:01 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/1a4850ccd8191d9d78d552a986c0162c519d9ab9

commit 1a4850ccd8191d9d78d552a986c0162c519d9ab9
Author:     Jason Frey <fryguy9@gmail.com>
AuthorDate: Tue Nov 22 12:58:28 2016 -0500
Commit:     Satoe Imaishi <simaishi@redhat.com>
CommitDate: Thu Jan 12 11:14:14 2017 -0500

    Merge pull request #12626 from Ladas/optimize_vm_and_generic_saving_code_by_removing_O_N_2
    
    Optimize vm and generic saving code by removing O(n^2)
    (cherry picked from commit d042596ca267fcb191fc96f9dbb8238d342e4692)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1412729

 app/models/ems_refresh/save_inventory.rb        | 143 +++++++++++++-----------
 app/models/ems_refresh/save_inventory_helper.rb |   4 +-
 2 files changed, 78 insertions(+), 69 deletions(-)

Comment 8 Greg Blomquist 2017-02-27 16:41:28 UTC
Pradeep,

not sure I understand comment #6.  Are you saying that this fix isn't in 5.7.1, but it *is* in a 5.7.2 build?

Comment 9 Greg Blomquist 2017-02-27 16:41:29 UTC
Pradeep,

not sure I understand comment #6.  Are you saying that this fix isn't in 5.7.1, but it *is* in a 5.7.2 build?


Note You need to log in before you can comment on or make changes to this bug.