Federico Simoncelli has uploaded a new change for review.
Change subject: spec: remove shared glusterfs requirements ......................................................................
spec: remove shared glusterfs requirements
Since we now require glusterfs in the main package we can now remove the same dependencies from the gluster rpm.
Change-Id: I6cdbf9a0bc97ce0bf42b0b7647a2641717442d58 Signed-off-by: Federico Simoncelli fsimonce@redhat.com --- M vdsm.spec.in 1 file changed, 0 insertions(+), 3 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/18408/1
diff --git a/vdsm.spec.in b/vdsm.spec.in index 74ad331..9666cc6 100644 --- a/vdsm.spec.in +++ b/vdsm.spec.in @@ -525,10 +525,7 @@ BuildArch: noarch
Requires: %{name} = %{version}-%{release} -Requires: glusterfs >= 3.4.0 Requires: glusterfs-server -Requires: glusterfs-fuse -Requires: glusterfs-rdma Requires: python-magic
%description gluster
oVirt Jenkins CI Server has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4081/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/3186/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/4002/ : SUCCESS
Deepak C Shetty has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1:
When we use a node as a gluster node (not as hypervisor/vdsm) i think it will only install vdsm-gluster, in which case we still would need the deps to be part of the vdsm-gluster pkg, no ?
I never tested on gluster only node.. Bala might be the right person to comment.
thanx, deepak
Deepak C Shetty has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1: Code-Review+1
vdsm-gluster indeed requires 'vdsm' main pkg.. so I am fine with this.
Douglas Schilling Landgraf has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1: Code-Review+1
Allon Mureinik has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1: Code-Review+1
Federico Simoncelli has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1: Verified+1
verified with:
$ rpm -qRp vdsm-gluster-4.12.0-102.gitde6f9a2.el6_4.noarch.rpm vdsm = 4.12.0-102.gitde6f9a2.el6_4 glusterfs-server python-magic ...
$ rpm -qRp vdsm-4.12.0-102.gitde6f9a2.el6_4.x86_64.rpm ... glusterfs >= 3.4.0 glusterfs-api glusterfs-cli glusterfs-fuse glusterfs-rdma ...
Dan Kenigsberg has posted comments on this change.
Change subject: spec: remove shared glusterfs requirements ......................................................................
Patch Set 1: Code-Review+2
Dan Kenigsberg has submitted this change and it was merged.
Change subject: spec: remove shared glusterfs requirements ......................................................................
spec: remove shared glusterfs requirements
Since we now require glusterfs in the main package we can now remove the same dependencies from the gluster rpm.
Change-Id: I6cdbf9a0bc97ce0bf42b0b7647a2641717442d58 Signed-off-by: Federico Simoncelli fsimonce@redhat.com Reviewed-on: http://gerrit.ovirt.org/18408 Reviewed-by: Deepak C Shetty deepakcs@linux.vnet.ibm.com Reviewed-by: Douglas Schilling Landgraf dougsland@redhat.com Reviewed-by: Allon Mureinik amureini@redhat.com Reviewed-by: Dan Kenigsberg danken@redhat.com --- M vdsm.spec.in 1 file changed, 0 insertions(+), 3 deletions(-)
Approvals: Douglas Schilling Landgraf: Looks good to me, but someone else must approve Federico Simoncelli: Verified Allon Mureinik: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Deepak C Shetty: Looks good to me, but someone else must approve
vdsm-patches@lists.fedorahosted.org