Bug 1610550

Summary:	Inventory Refresh taking 10-14 mins per migrated vm
Product:	Red Hat CloudForms Management Engine	Reporter:	Satoe Imaishi <simaishi>
Component:	Providers	Assignee:	Fabien Dupont <fdupont>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Dave Johnson <dajohnso>
Severity:	medium	Docs Contact:
Priority:	high
Version:	5.9.4	CC:	bthurber, dagur, dmetzger, fdupont, gblomqui, jfrey, jhardy, jprause, mlehrer, obarenbo, simaishi, smallamp
Target Milestone:	GA	Keywords:	ZStream
Target Release:	5.9.4
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	5.9.4.4	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1607368	Environment:
Last Closed:	2018-09-05 12:55:38 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:	1607368
Bug Blocks:

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