Change in vdsm[ovirt-3.3]: vdsm.spec: BZ 988299: Fix GlusterFS RPM dep to support Glust...

fsimonce at redhat.com fsimonce at redhat.com
Thu Aug 22 15:21:19 UTC 2013


Hello Bala.FA, Deepak C Shetty,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/18426

to review the following change.

Change subject: vdsm.spec: BZ 988299: Fix GlusterFS RPM dep to support GlusterFS SD
......................................................................

vdsm.spec: BZ 988299: Fix GlusterFS RPM dep to support GlusterFS SD

This patch fixes the GlusterFS RPM dependency needed by VDSM to
support Gluster Storage Domain feature. Recent GlusterFS RPM
re-packaging effort to better separate the client and server side
bits and availability of qemu with libgfapi support mark the
pre-req for this patch. These pre-req are now available in
F18/F19/RHEL6/Centos6 and above in the appropriate distro specific
repos. Also removing F17 dep as its End of life now.

BZ 988299 has more details, with pointers to the community
discussions around these.

Change-Id: If648feec92ed33f24f45550f016ac132a25e6923
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=988299
Signed-off-by: Deepak C Shetty <deepakcs at linux.vnet.ibm.com>
Reviewed-on: http://gerrit.ovirt.org/17994
Reviewed-by: Bala.FA <barumuga at redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm.spec.in
1 file changed, 10 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/26/18426/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index 21b3565..74ad331 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -25,7 +25,7 @@
 %global _polkitdir %{_localstatedir}/lib/polkit-1/localauthority/10-vendor.d
 %endif
 
-%if 0%{?fedora} >= 17 || 0%{?rhel} >= 6
+%if 0%{?fedora} >= 18 || 0%{?rhel} >= 6
 %global with_gluster 1
 %endif
 
@@ -198,6 +198,15 @@
 Requires: systemd >= 197-1.fc18.2
 %endif
 
+# GlusterFS client-side RPMs needed for Gluster SD
+%if 0%{?with_gluster}
+Requires: glusterfs >= 3.4.0
+Requires: glusterfs-cli
+Requires: glusterfs-api
+Requires: glusterfs-fuse
+Requires: glusterfs-rdma
+%endif
+
 Requires: psmisc >= 22.6-15
 Requires: bridge-utils
 Requires: sos


-- 
To view, visit http://gerrit.ovirt.org/18426
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If648feec92ed33f24f45550f016ac132a25e6923
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Deepak C Shetty <deepakcs at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list