Bug 1610550 - Inventory Refresh taking 10-14 mins per migrated vm
Summary: Inventory Refresh taking 10-14 mins per migrated vm
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.9.4
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: GA
: 5.9.4
Assignee: Fabien Dupont
QA Contact: Dave Johnson
URL:
Whiteboard:
Depends On: 1607368
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-07-31 21:51 UTC by Satoe Imaishi
Modified: 2018-09-05 12:55 UTC (History)
12 users (show)

Fixed In Version: 5.9.4.4
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1607368
Environment:
Last Closed: 2018-09-05 12:55:38 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github patternfly patternfly issues 1096 0 None None None 2018-07-31 21:51:49 UTC

Comment 2 CFME Bot 2018-07-31 21:53:53 UTC
New commit detected on ManageIQ/manageiq-content/gaprindashvili:

https://github.com/ManageIQ/manageiq-content/commit/f559300aad804be604f5b8bc3f5f9fd0a22e36dd
commit f559300aad804be604f5b8bc3f5f9fd0a22e36dd
Author:     Greg McCullough <gmccullo>
AuthorDate: Mon Jul 30 15:33:29 2018 -0400
Commit:     Greg McCullough <gmccullo>
CommitDate: Mon Jul 30 15:33:29 2018 -0400

    Merge pull request #381 from fdupont-redhat/v2v_use_targeted_refresh

    Use targeted refresh during transformation
    (cherry picked from commit 4a918271d51c80cfc56941501754249b9ef1767a)

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

 content/automate/ManageIQ/System/Event/EmsEvent/RHEVM.class/vm_import_from_configuration_executed_successfully.yaml | 12 +
 content/automate/ManageIQ/Transformation/Infrastructure/VM/rhevm.class/__methods__/checkvmininventory.rb | 20 +-
 2 files changed, 15 insertions(+), 17 deletions(-)

Comment 4 Mor 2018-08-09 04:25:26 UTC
Behaviour seems as expected, however, we still have functional issues on our environment with multi-host cases, so I haven't got a chance to look at a complete successful run. I'll verify this bug, and reopen in case needed.

Verified on:
CFME 5.9.4.2
RHV 4.2.5.2-0.1.el7ev
VMWare 6.5

Comment 5 Satoe Imaishi 2018-08-13 16:35:28 UTC
Extra PR - Remove check on number of runners when converting VM
https://github.com/ManageIQ/manageiq-content/pull/395

Comment 6 CFME Bot 2018-08-13 16:38:14 UTC
New commit detected on ManageIQ/manageiq-content/gaprindashvili:

https://github.com/ManageIQ/manageiq-content/commit/17ab5dac50196eb5f58ade4c891856477103b236
commit 17ab5dac50196eb5f58ade4c891856477103b236
Author:     Madhu Kanoor <mkanoor>
AuthorDate: Thu Aug  9 11:26:00 2018 -0400
Commit:     Madhu Kanoor <mkanoor>
CommitDate: Thu Aug  9 11:26:00 2018 -0400

    Merge pull request #395 from fdupont-redhat/v2v_fix_workers_balance

    Remove check on number of runners when converting VM
    (cherry picked from commit 7a0d04d09e2f66cb815aa51af5bdc0475930f56d)

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

 content/automate/ManageIQ/Transformation/TransformationHosts/Common.class/__methods__/vmtransform.rb | 40 +-
 1 file changed, 13 insertions(+), 27 deletions(-)

Comment 8 Mor 2018-08-20 06:12:15 UTC
@Satoe, I wonder what should I verify after the extra PR?

Comment 9 Satoe Imaishi 2018-08-20 13:32:53 UTC
As per https://bugzilla.redhat.com/show_bug.cgi?id=1607368#c11

"The previously mentioned PR was incorrectly linked and does not belong to this BZ."

Comment 10 Mor 2018-08-20 18:59:34 UTC
Verified on:
CFME 5.9.4.4.20180816162527_c00eb23


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