Bala.FA has posted comments on this change.

Change subject: Gluster packages are not always available
......................................................................


Patch Set 3: Code-Review-1

(2 comments)

https://gerrit.ovirt.org/#/c/41645/3/vdsm.spec.in
File vdsm.spec.in:

Line 294
Line 295
Line 296
Line 297
Line 298
> For mounting gluster storage domain and use gluster based vm disk, we must 
+1

better to have

Requires: glusterfs-cli >= 3.6.999


Line 296
Line 297
Line 298
Line 299
Line 300
> To mount glusterfs storage domain, we must have /usr/sbin/mount.glusterfs. 
+1

glusterfs-fuse >= 3.6.999

please note that vdsm-gluster sub-package should depend on glusterfs-api


-- 
To view, visit https://gerrit.ovirt.org/41645
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia51d9c30538c8f5f6f287c18de8fc2803ac1c76d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Bala.FA <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to