Bug 1259282

Summary:	vdsm rpms are built with wrong prefix
Product:	[oVirt] vdsm	Reporter:	Sandro Bonazzola <sbonazzo>
Component:	General	Assignee:	Anton Marchukov <amarchuk>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Jiri Belka <jbelka>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	---	CC:	bazulay, bugs, dcaroest, eedri, gklein, lsurette, mgoldboi, rbalakri, ycui, yeylon, ykaul
Target Milestone:	ovirt-3.6.0-rc	Keywords:	Regression
Target Release:	4.17.8	Flags:	rule-engine: ovirt-3.6.0+ rule-engine: blocker+ ylavi: planning_ack+ rule-engine: devel_ack+ pstehlik: testing_ack+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-02-10 12:58:25 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Integration	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Sandro Bonazzola 2015-09-02 11:04:58 UTC

3.6 rpms are built with wrong prefix (/usr/local instead of /usr)

See as example src.rpm from

 http://jenkins.ovirt.org/job/vdsm_3.6_create-rpms-el7-x86_64_merged/31/
 http://jenkins.ovirt.org/job/vdsm_3.6_create-rpms-fc21-x86_64_merged/30/
 http://jenkins.ovirt.org/job/vdsm_3.6_create-rpms-fc22-x86_64_merged/27/

Comment 1 Sandro Bonazzola 2015-09-02 11:08:34 UTC

Moving to amarchuk since he's infra owner this week

Comment 2 Anton Marchukov 2015-09-02 12:00:52 UTC

We added the flags to configure, but for some reason with current spec it is then passed to install:

+ install -dDm 0755 /builddir/build/BUILDROOT/vdsm-4.16.26-0.fc21.x86_64/usr --sysconfdir=/etc --localstatedir=/var --libdir=/usr/lib64/var/log/vdsm
install: unrecognized option '--sysconfdir=/etc'
Try 'install --help' for more information.
error: Bad exit status from /var/tmp/rpm-tmp.VxOvvS (%install)
RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.VxOvvS (%install)
Child return code was: 1

http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc21-x86_64_merged/128/artifact/exported-artifacts/build.log
https://gerrit.ovirt.org/#/c/45079/

Comment 3 Anton Marchukov 2015-09-02 16:11:21 UTC

The patch is ready:

1. First https://gerrit.ovirt.org/#/c/45674/ to fix mox extra configure options passing
2. Second sbonazzo change https://gerrit.ovirt.org/#/c/45079/

Comment 4 Red Hat Bugzilla Rules Engine 2015-09-22 07:43:59 UTC

This bug report has Keywords: Regression or TestBlocker.
Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP.

Comment 5 Jiri Belka 2016-01-21 08:54:13 UTC

ok, vdsm-4.17.17-6.git37bda50.el7.centos.noarch.rpm

[root@dell-r210ii-04 tmp]# rpm -qlp http://resources.ovirt.org/pub/ovirt-3.6-snapshot/rpm/el7/noarch/vdsm-4.17.17-6.git37bda50.el7.centos.noarch.rpm  | grep /usr/local
[root@dell-r210ii-04 tmp]#