Bug 1349529 - VDSM to require qemu-kvm-ev delivered by bug 1349525
Summary: VDSM to require qemu-kvm-ev delivered by bug 1349525
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: vdsm
Classification: oVirt
Component: Build
Version: 4.17.32
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ovirt-4.0.2
: 4.18.10
Assignee: Allon Mureinik
QA Contact: Aharon Canan
URL:
Whiteboard:
Depends On: 1349525
Blocks: 1361552
TreeView+ depends on / blocked
 
Reported: 2016-06-23 15:40 UTC by Allon Mureinik
Modified: 2016-08-12 14:31 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
: 1361552 (view as bug list)
Environment:
Last Closed: 2016-08-12 14:31:56 UTC
oVirt Team: Storage
Embargoed:
rule-engine: ovirt-4.0.z+
ylavi: Triaged+
ylavi: planning_ack+
rule-engine: devel_ack+
acanan: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 60859 0 master MERGED spec: Require qemu-kvm-ev to fix aio_bh_poll hang 2016-07-18 12:40:09 UTC
oVirt gerrit 60931 0 ovirt-4.0 MERGED spec: Require qemu-kvm-ev to fix aio_bh_poll hang 2016-07-19 07:04:12 UTC
oVirt gerrit 60932 0 ovirt-3.6 MERGED spec: Require qemu-kvm-ev to fix aio_bh_poll hang 2016-07-19 07:16:07 UTC
oVirt gerrit 62104 0 master MERGED spec: qemu: add _2 to %dist 2016-08-09 08:57:03 UTC

Description Allon Mureinik 2016-06-23 15:40:25 UTC
Description of problem:
Bug 1349525 describes a qemu-kvm-ev build that will provide several fixes around live merging.
VDSM should require this version to make sure it's delivered to oVirt users.

Comment 1 Sandro Bonazzola 2016-07-29 11:17:59 UTC
3.6 is gone EOL; Please re-target this bug to a 4.0 release.

Comment 2 Allon Mureinik 2016-07-31 09:33:51 UTC
(In reply to Sandro Bonazzola from comment #1)
> 3.6 is gone EOL; Please re-target this bug to a 4.0 release.
Retargetting. The 4.0.z patch was included in vdsm-4.18.7

Comment 3 Red Hat Bugzilla Rules Engine 2016-07-31 09:33:56 UTC
Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.

Comment 4 Red Hat Bugzilla Rules Engine 2016-07-31 09:34:27 UTC
Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.

Comment 5 Aharon Canan 2016-08-02 08:16:29 UTC
[root@storage-ge9-vdsm1 ~]# yum deplist vdsm-4.18.9-1.el7ev.x86_64 |grep qemu
Failed to set locale, defaulting to C
  dependency: qemu-img-rhev >= 10:2.3.0-31.el7_2.16
   provider: qemu-img-rhev.x86_64 10:2.3.0-31.el7_2.20
  dependency: qemu-kvm-rhev >= 10:2.3.0-31.el7_2.16
   provider: qemu-kvm-rhev.x86_64 10:2.3.0-31.el7_2.20

Comment 6 Markus Stockhausen 2016-08-06 06:50:42 UTC
This patch seems to totally break with the qemu from OVirt repos http://resources.ovirt.org/pub/ovirt-4.0/rpm/el7Server/x86_64/

qemu version changed from el7_2.10.1 to el7.16.1. Notice the 2 is missing. Thus the following check is quite confusing


Requires: qemu-img-rhev >= 10:2.3.0-31.el7_2.16

Comment 7 Allon Mureinik 2016-08-08 11:02:09 UTC
(In reply to Markus Stockhausen from comment #6)
> This patch seems to totally break with the qemu from OVirt repos
> http://resources.ovirt.org/pub/ovirt-4.0/rpm/el7Server/x86_64/
> 
> qemu version changed from el7_2.10.1 to el7.16.1. Notice the 2 is missing.
> Thus the following check is quite confusing
> 
> 
> Requires: qemu-img-rhev >= 10:2.3.0-31.el7_2.16

Sandro - the patch follows the naming conventions from the previous version and from RHEL.
IIUC, the rpm in the repo is misnamed. Can you take a look please?

Comment 8 Sandro Bonazzola 2016-08-09 08:23:41 UTC
(In reply to Allon Mureinik from comment #7)
> (In reply to Markus Stockhausen from comment #6)
> > This patch seems to totally break with the qemu from OVirt repos
> > http://resources.ovirt.org/pub/ovirt-4.0/rpm/el7Server/x86_64/
> > 
> > qemu version changed from el7_2.10.1 to el7.16.1. Notice the 2 is missing.
> > Thus the following check is quite confusing
> > 
> > 
> > Requires: qemu-img-rhev >= 10:2.3.0-31.el7_2.16
> 
> Sandro - the patch follows the naming conventions from the previous version
> and from RHEL.
> IIUC, the rpm in the repo is misnamed. Can you take a look please?

I'll try to rebuild with %dist set to .el7_2; honestly I don't understand why the _2 is added there in the original rpm.


Note You need to log in before you can comment on or make changes to this bug.