Bug 1352967

Summary:	Inventory refresh doesn't work with version 4 of oVirt
Product:	Red Hat CloudForms Management Engine	Reporter:	Juan Hernández <juan.hernandez>
Component:	Providers	Assignee:	Juan Hernández <juan.hernandez>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ilanit Stein <istein>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	5.5.0	CC:	istein, jfrey, jhardy, jprause, obarenbo, oourfali, simaishi
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.7.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	rhev
Fixed In Version:	5.7.0.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1353323 (view as bug list)		Environment:
Last Closed:	2017-01-11 20:14:09 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1353323, 1353720

Description Juan Hernández 2016-07-05 15:36:26 UTC

Description of problem:

When version 4 or later of oVirt (or RHEV-M) is added to CFME as a provider, inventory refresh doesn't work correctly.


Version-Release number of selected component (if applicable):

Any version of CFME.
Version 4 or newer of oVirt or RHEV-M.

How reproducible:

Always.


Steps to Reproduce:

1. Add the oVirt provider.
2. Wait for the inventory refresh.

Actual results:

The refresh fails and the log of the web server in the oVirt machine shows requests that start with the following incorrect prefix:

  /ovirt-engine/api/ovirt-engine/api

Expected results:

The refresh should work correctly.

Comment 1 Juan Hernández 2016-07-05 15:42:27 UTC

The following PR are required to fix this issue:

  Remove "ovirt-engine" prefix from paths
  https://github.com/ManageIQ/ovirt/pull/55

  Don't add the /ovirt-engine prefix to oVirt ems_ref
  https://github.com/ManageIQ/manageiq/pull/9457

In addition, once the change to the `ovirt` gem is merged, it needs to be released, and an additional PR request will need to be done to use the new version.

Comment 3 Juan Hernández 2016-07-11 13:41:47 UTC

The fix to update the version of the `ovirt` gem has already been merged to the master branch, as part of this pull request:

  Fix gem issues related to rails 5.0 release
  https://github.com/ManageIQ/manageiq/pull/9667

So this bug can now be moved to MODIFIED.

Comment 4 Ilanit Stein 2016-10-05 13:21:23 UTC

Verified on CFME-5.7.0.3.20160927165516_075d0f3
and RHV-4.0.4.2-0.1.el7ev