Change in vdsm[master]: Removed glusterfs packages from bootstrap

barumuga at redhat.com barumuga at redhat.com
Tue Nov 6 11:48:52 UTC 2012


Hello Ayal Baron, Timothy Asir, Saggi Mizrahi, Federico Simoncelli, Dan Kenigsberg,

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

    http://gerrit.ovirt.org/9076

to review the following change.

Change subject: Removed glusterfs packages from bootstrap
......................................................................

Removed glusterfs packages from bootstrap

As glusterfs packages are dependencies for vdsm-gluster package,
having these in bootstrap is redundant

Change-Id: I5bdde338599155df43af3a0fd0d14a02e3bddda8
Signed-off-by: Bala.FA <barumuga at redhat.com>
---
M vds_bootstrap/vds_bootstrap.py
M vdsm.spec.in
2 files changed, 2 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/76/9076/1

diff --git a/vds_bootstrap/vds_bootstrap.py b/vds_bootstrap/vds_bootstrap.py
index 867d6f4..95f255d 100755
--- a/vds_bootstrap/vds_bootstrap.py
+++ b/vds_bootstrap/vds_bootstrap.py
@@ -159,8 +159,7 @@
                 'seabios', 'qemu-img', 'fence-agents',
                 'libselinux-python', 'sanlock', 'sanlock-python')
     # Gluster packages
-    GLUSTER_PACK = ('vdsm-gluster', 'glusterfs-server', 'glusterfs-rdma',
-                    'glusterfs-geo-replication')
+    GLUSTER_PACK = (VDSM_NAME+'-gluster')
 else:
     # Devel packages
     DEVEL_PACK = ('gdb','tcpdump','strace','ltrace','sysstat','ntp',
diff --git a/vdsm.spec.in b/vdsm.spec.in
index c117ee5..8070cbe 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -356,7 +356,7 @@
 BuildArch:      noarch
 
 Requires: %{name} = %{version}-%{release}
-Requires: glusterfs glusterfs-server glusterfs-fuse
+Requires: glusterfs, glusterfs-server, glusterfs-fuse, glusterfs-rdma
 
 %description gluster
 Gluster plugin enables VDSM to serve Gluster functionalities.


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5bdde338599155df43af3a0fd0d14a02e3bddda8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>


More information about the vdsm-patches mailing list