Nir Soffer has posted comments on this change.

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


Patch Set 3: Code-Review-1

(2 comments)

This patch will break glusterfs storage domain as is; see the comments in the 
spec.

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 have 
/usr/sbin/gluster command line on the local machine. This command is provided 
by glusterfs-cli, so this package must be required.


Line 296
Line 297
Line 298
Line 299
Line 300
To mount glusterfs storage domain, we must have /usr/sbin/mount.glusterfs. This 
command is provided by glusterfs-fuse package, so this package must be required.


-- 
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: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[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