Timothy Asir has posted comments on this change.
Change subject: Caps: Include glusterfs packages ......................................................................
Patch Set 4: No score
(2 inline comments)
.................................................... File vdsm/caps.py Line 329: pkgs = {'kernel': kernelDict()} Line 330: Line 331: if getos() in (OSName.RHEVH, OSName.OVIRT, OSName.FEDORA, OSName.RHEL): Line 332: GLUSTER_PACKAGES = ['glusterfs', 'glusterfs-fuse', Line 333: 'glusterfs-geo-replication', 'glusterfs-rdma', 1) yes, we need to track all the packages to provide gluster capabilities like one can use fuse-mount , rdma, geo-replication, swift, etc.
2) yes, I will sort them and keep the same order in both places. Line 334: 'glusterfs-server', 'glusterfs-swift', Line 335: 'glusterfs-swift-account', Line 336: 'glusterfs-swift-container', 'glusterfs-swift-doc', Line 337: 'glusterfs-swift-object', 'glusterfs-swift-proxy',
Line 361: GLUSTER_PACKAGES = { Line 362: 'gluster-swift': 'glusterfs-swift', Line 363: 'gluster-swift-account': 'glusterfs-swift-account', Line 364: 'gluster-swift-container': 'glusterfs-swift-container', Line 365: 'gluster-swift-doc': 'glusterfs-swift-doc', I am still evaluating this. I will send a patch once its done. Line 366: 'gluster-swift-object': 'glusterfs-swift-object', Line 367: 'gluster-swift-plugin': 'glusterfs-ufo', Line 368: 'gluster-swift-proxy': 'glusterfs-swift-proxy', Line 369: 'glusterfs': 'glusterfs',
-- To view, visit http://gerrit.ovirt.org/11138 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I38991550e57c8f1e558f297cf12d984f7f0bc635 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Timothy Asir tjeyasin@redhat.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Bala.FA barumuga@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Timothy Asir tjeyasin@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server