Bug 1691109

Summary: RHV provider is recreating guest_devices on every refresh
Product: Red Hat CloudForms Management Engine Reporter: Jared Deubel <jdeubel>
Component: ProvidersAssignee: Boriso <bodnopoz>
Status: CLOSED CURRENTRELEASE QA Contact: Angelina Vasileva <anikifor>
Severity: medium Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.10.0CC: agrare, anikifor, dmetzger, jfrey, jhardy, jocarter, mperina, mshriver, obarenbo, rspagnol, simaishi, smallamp
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.11.0Flags: mshriver: needinfo-
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.11.0.2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1731237 (view as bug list) Environment:
Last Closed: 2019-12-13 14:57:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: RHEVM Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1731237    

Comment 6 CFME Bot 2019-04-25 16:37:23 UTC
New commit detected on ManageIQ/manageiq-providers-ovirt/master:

https://github.com/ManageIQ/manageiq-providers-ovirt/commit/d4f29038eeab1fd769231e3c1c040ce1c5b3d897
commit d4f29038eeab1fd769231e3c1c040ce1c5b3d897
Author:     Adam Grare <agrare>
AuthorDate: Thu Apr 25 09:31:35 2019 -0400
Commit:     Adam Grare <agrare>
CommitDate: Thu Apr 25 09:31:35 2019 -0400

    Use the host_geust_devices inventory collection

    Prevent duplication of host guest_devices by using the correct
    association.

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

 app/models/manageiq/providers/redhat/inventory/parser/infra_manager.rb | 2 +-
 app/models/manageiq/providers/redhat/inventory/persister/definitions/infra_collections.rb | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

Comment 7 Angelina Vasileva 2019-05-06 09:35:13 UTC
Fixed and verified in 5.11.0.2.20190430174828_0e34dea.

GuestDevice.count stays the same after the refresh.

Comment 10 Sudhir Mallamprabhakara 2019-05-10 14:06:23 UTC
@Mike - Can you review this in next Triage?