Bug 1349529

Summary: VDSM to require qemu-kvm-ev delivered by bug 1349525
Product: [oVirt] vdsm Reporter: Allon Mureinik <amureini>
Component: BuildAssignee: Allon Mureinik <amureini>
Status: CLOSED CURRENTRELEASE QA Contact: Aharon Canan <acanan>
Severity: high Docs Contact:
Priority: unspecified    
Version: 4.17.32CC: acanan, bugs, mst, sbonazzo, ylavi
Target Milestone: ovirt-4.0.2Flags: rule-engine: ovirt-4.0.z+
ylavi: Triaged+
ylavi: planning_ack+
rule-engine: devel_ack+
acanan: testing_ack+
Target Release: 4.18.10   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1361552 (view as bug list) Environment:
Last Closed: 2016-08-12 14:31:56 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1349525    
Bug Blocks: 1361552    

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.