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 <tjeya...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Bala.FA <barum...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeya...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to