Broken dependency for VDSM

Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20 Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389 -- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com

On Thu, Mar 19, 2015 at 09:00:27AM +0100, Sandro Bonazzola wrote:
Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20
Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389
The faulty patch is gluster: add createBrick verb https://gerrit.ovirt.org/35498 Timothy, please make sure that Vdsm builds again on f20. The bare minimum is -%if 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhelf 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhel} > 6 || 0%{?fedora} > 20 BuildRequires: python-blivet >= 0.61.14 Though you must also report to Engine in runtime whether you have brick-adding feature enabled or not, based on runtime existence of blivet.

----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "Sandro Bonazzola" <sbonazzo@redhat.com>, "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com> Cc: devel@ovirt.org Sent: Thursday, March 19, 2015 12:06:22 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
On Thu, Mar 19, 2015 at 09:00:27AM +0100, Sandro Bonazzola wrote:
Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20
Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389
The faulty patch is
gluster: add createBrick verb https://gerrit.ovirt.org/35498
Timothy, please make sure that Vdsm builds again on f20. The bare minimum is
-%if 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhelf 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhel} > 6 || 0%{?fedora} > 20 BuildRequires: python-blivet >= 0.61.14
Please also note that centos 7 ships python-blivet-0.18.34-1.el7.noarch Bests, -- Francesco Romani RedHat Engineering Virtualization R & D Phone: 8261328 IRC: fromani

Il 19/03/2015 12:08, Francesco Romani ha scritto:
----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "Sandro Bonazzola" <sbonazzo@redhat.com>, "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com> Cc: devel@ovirt.org Sent: Thursday, March 19, 2015 12:06:22 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
On Thu, Mar 19, 2015 at 09:00:27AM +0100, Sandro Bonazzola wrote:
Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20
Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389
The faulty patch is
gluster: add createBrick verb https://gerrit.ovirt.org/35498
Timothy, please make sure that Vdsm builds again on f20. The bare minimum is
-%if 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhelf 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhel} > 6 || 0%{?fedora} > 20 BuildRequires: python-blivet >= 0.61.14
Please also note that centos 7 ships
python-blivet-0.18.34-1.el7.noarch
CentOS 7.1 deliver 0.61.0.26 so it's not satisfied in CentOS 7.1 as well. http://mirror.centos.org/centos/7/cr/x86_64/Packages/python-blivet-0.61.0.26...
Bests,
-- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com

Blivet returns the unit size as an object (blivet.size) in blivet version >= 0.61.14 which is available in F21. We thought that using size object will be more clearer; because it provides unit type and some more functionality to change from one unit to another easily. But unfortunately the lower version in F20 just return the size in MiB (not an object). I will fix this issue by sending a patch to vdsm to check the unit type of the device and fetch the value accordingly and remove any buildRequire version for the python-blivet package in the spec file. So that it can be build and work in F20 Regards, Tim ----- Original Message -----
From: "Sandro Bonazzola" <sbonazzo@redhat.com> To: "Francesco Romani" <fromani@redhat.com>, "Dan Kenigsberg" <danken@redhat.com> Cc: "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com>, devel@ovirt.org Sent: Thursday, March 19, 2015 5:17:15 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
Il 19/03/2015 12:08, Francesco Romani ha scritto:
----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "Sandro Bonazzola" <sbonazzo@redhat.com>, "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com> Cc: devel@ovirt.org Sent: Thursday, March 19, 2015 12:06:22 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
On Thu, Mar 19, 2015 at 09:00:27AM +0100, Sandro Bonazzola wrote:
Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20
Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389
The faulty patch is
gluster: add createBrick verb https://gerrit.ovirt.org/35498
Timothy, please make sure that Vdsm builds again on f20. The bare minimum is
-%if 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhelf 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhel} > 6 || 0%{?fedora} > 20 BuildRequires: python-blivet >= 0.61.14
Please also note that centos 7 ships
python-blivet-0.18.34-1.el7.noarch
CentOS 7.1 deliver 0.61.0.26 so it's not satisfied in CentOS 7.1 as well. http://mirror.centos.org/centos/7/cr/x86_64/Packages/python-blivet-0.61.0.26...
Bests,
-- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com

On 03/19/2015 02:17 PM, Timothy Asir Jeyasingh wrote:
Blivet returns the unit size as an object (blivet.size) in blivet version >= 0.61.14 which is available in F21. We thought that using size object will be more clearer; because it provides unit type and some more functionality to change from one unit to another easily. But unfortunately the lower version in F20 just return the size in MiB (not an object).
I will fix this issue by sending a patch to vdsm to check the unit type of the device and fetch the value accordingly and remove any buildRequire version for the python-blivet package in the spec file. So that it can be build and work in F20
please do that asap or accept the revert and push it back after you have this fix.. we can't keep it that way
Regards, Tim
----- Original Message -----
From: "Sandro Bonazzola" <sbonazzo@redhat.com> To: "Francesco Romani" <fromani@redhat.com>, "Dan Kenigsberg" <danken@redhat.com> Cc: "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com>, devel@ovirt.org Sent: Thursday, March 19, 2015 5:17:15 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
Il 19/03/2015 12:08, Francesco Romani ha scritto:
----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "Sandro Bonazzola" <sbonazzo@redhat.com>, "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com> Cc: devel@ovirt.org Sent: Thursday, March 19, 2015 12:06:22 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
On Thu, Mar 19, 2015 at 09:00:27AM +0100, Sandro Bonazzola wrote:
Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20
Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389
The faulty patch is
gluster: add createBrick verb https://gerrit.ovirt.org/35498
Timothy, please make sure that Vdsm builds again on f20. The bare minimum is
-%if 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhelf 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhel} > 6 || 0%{?fedora} > 20 BuildRequires: python-blivet >= 0.61.14
Please also note that centos 7 ships
python-blivet-0.18.34-1.el7.noarch
CentOS 7.1 deliver 0.61.0.26 so it's not satisfied in CentOS 7.1 as well. http://mirror.centos.org/centos/7/cr/x86_64/Packages/python-blivet-0.61.0.26...
Bests,
-- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com
_______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
-- Yaniv Bronhaim.

Patch sent to upstream: https://gerrit.ovirt.org/#/c/38942/ I will test it in F20 and F21 and verify the patch by today. Regards, Tim ----- Original Message -----
From: "ybronhei" <ybronhei@redhat.com> To: "Timothy Asir Jeyasingh" <tjeyasin@redhat.com>, "Sandro Bonazzola" <sbonazzo@redhat.com> Cc: devel@ovirt.org Sent: Thursday, March 19, 2015 6:24:10 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
On 03/19/2015 02:17 PM, Timothy Asir Jeyasingh wrote:
= 0.61.14 which is available in F21. We thought that using size object will be more clearer; because it provides unit type and some more functionality to change from one unit to another easily. But unfortunately
Blivet returns the unit size as an object (blivet.size) in blivet version the lower version in F20 just return the size in MiB (not an object).
I will fix this issue by sending a patch to vdsm to check the unit type of the device and fetch the value accordingly and remove any buildRequire version for the python-blivet package in the spec file. So that it can be build and work in F20
please do that asap or accept the revert and push it back after you have this fix.. we can't keep it that way
Regards, Tim
----- Original Message -----
From: "Sandro Bonazzola" <sbonazzo@redhat.com> To: "Francesco Romani" <fromani@redhat.com>, "Dan Kenigsberg" <danken@redhat.com> Cc: "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com>, devel@ovirt.org Sent: Thursday, March 19, 2015 5:17:15 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
Il 19/03/2015 12:08, Francesco Romani ha scritto:
----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "Sandro Bonazzola" <sbonazzo@redhat.com>, "Timothy Asir" <tjeyasin@redhat.com>, "Bala.FA" <barumuga@redhat.com> Cc: devel@ovirt.org Sent: Thursday, March 19, 2015 12:06:22 PM Subject: Re: [ovirt-devel] Broken dependency for VDSM
On Thu, Mar 19, 2015 at 09:00:27AM +0100, Sandro Bonazzola wrote:
Hi, a recent change introduced a dependency on python-blivet >= 0.61.14 which is not available at least on Fedora 20
Please provide the required package or ask fedora maintainer to backport to Fedora 20. http://koji.fedoraproject.org/koji/packageinfo?packageID=15389
The faulty patch is
gluster: add createBrick verb https://gerrit.ovirt.org/35498
Timothy, please make sure that Vdsm builds again on f20. The bare minimum is
-%if 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhelf 0%{?rhel} > 6 || 0%{?fedora} +%if 0%{?rhel} > 6 || 0%{?fedora} > 20 BuildRequires: python-blivet >= 0.61.14
Please also note that centos 7 ships
python-blivet-0.18.34-1.el7.noarch
CentOS 7.1 deliver 0.61.0.26 so it's not satisfied in CentOS 7.1 as well. http://mirror.centos.org/centos/7/cr/x86_64/Packages/python-blivet-0.61.0.26...
Bests,
-- Sandro Bonazzola Better technology. Faster innovation. Powered by community collaboration. See how it works at redhat.com
_______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
-- Yaniv Bronhaim.
participants (5)
-
Dan Kenigsberg
-
Francesco Romani
-
Sandro Bonazzola
-
Timothy Asir Jeyasingh
-
ybronhei