Bug 1055517

Summary:	log-collector accesses /api instead of /ovirt-engine/api
Product:	[Retired] oVirt	Reporter:	Yedidyah Bar David <didi>
Component:	ovirt-engine-logcollector	Assignee:	Sandro Bonazzola <sbonazzo>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Petr Beňas <pbenas>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	3.3	CC:	acathrow, alonbl, gklein, iheim, oschreib, pstehlik, sbonazzo, talayan, yeylon
Target Milestone:	---	Keywords:	EasyFix
Target Release:	3.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	integration
Fixed In Version:	ovirt-3.4.0-beta3	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-03-31 12:32:06 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:

Description Yedidyah Bar David 2014-01-20 13:15:50 UTC

Description of problem:

Following bz 961677 the REST API should be accessed using /ovirt-engine/api and not /api.

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

3.4

How reproducible:

Always

Steps to Reproduce:
1. engine-log-collector collect
2.
3.

Actual results:

Fails connecting to the engine with:
ERROR: Unable to connect to REST API.  Reason: Bad Request

Expected results:

Should not fail

Additional info:

Comment 1 Alon Bar-Lev 2014-01-20 13:20:04 UTC

but we do support both /api and /ovirt-engine/api if there is a problem we need to know what it is.

anyway, stupid question... why is log collector using the api?

Comment 2 Yedidyah Bar David 2014-01-20 14:38:01 UTC

(In reply to Alon Bar-Lev from comment #1)
> but we do support both /api and /ovirt-engine/api if there is a problem we
> need to know what it is.

Right, sorry. There is no problem. Still we might want to change it to use /ovirt-engine/api, but it currently fails for me for some other reason. I'll check and might open another bug.

> 
> anyway, stupid question... why is log collector using the api?

To get a list of the hosts to collect logs from.

Comment 3 Alon Bar-Lev 2014-01-20 14:41:44 UTC

(In reply to Yedidyah Bar David from comment #2)
> > 
> > anyway, stupid question... why is log collector using the api?
> 
> To get a list of the hosts to collect logs from.

hmmm.... won't it easier to access the database directly? then it can run if engine is down and no need for application admin password...

Comment 4 Itamar Heim 2014-01-26 08:11:36 UTC

Setting target release to current version for consideration and review. please
do not push non-RFE bugs to an undefined target release to make sure bugs are
reviewed for relevancy, fix, closure, etc.

Comment 5 Sandro Bonazzola 2014-02-06 12:40:52 UTC

merged on upstream master and 3.4.0 branch.

Comment 6 Petr Beňas 2014-02-25 12:27:24 UTC

Basic sanity test. Verified engine-log-collector collect works in ovirt-log-collector-3.4.0-0.5.beta3.el6.noarch.

Comment 7 Sandro Bonazzola 2014-03-31 12:32:06 UTC

this is an automated message: moving to Closed CURRENT RELEASE since oVirt 3.4.0 has been released