Change in vdsm[master]: Caps: Include glusterfs packages
danken at redhat.com
danken at redhat.com
Mon Mar 4 18:51:17 UTC 2013
Dan Kenigsberg has posted comments on this change.
Change subject: Caps: Include glusterfs packages
......................................................................
Patch Set 5: (1 inline comment)
....................................................
File vdsm/caps.py
Line 372: elif getos() == OSName.DEBIAN and python_apt:
Line 373: KEY_PACKAGES = {'qemu-kvm': 'qemu-kvm', 'qemu-img': 'qemu-utils',
Line 374: 'vdsm': 'vdsmd', 'spice-server': 'libspice-server1',
Line 375: 'libvirt': 'libvirt0', 'mom': 'mom'}
Line 376: GLUSTER_PACKAGES = {'glusterfs-client': 'glusterfs-client',
I don't get it - you do not have the same keys here and in the rpm-based part. We'd like this to be a portability layer, so that Engine should not care if it's RPM or DEB.
Maybe the mapping is impossible - but it does not seem that there's an attempt here.
Line 377: 'glusterfs-common': 'glusterfs-common',
Line 378: 'glusterfs-dbg': 'glusterfs-dbg',
Line 379: 'glusterfs-server': 'glusterfs-server',
Line 380: 'libglusterfs-dev': 'libglusterfs-dev',
--
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: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list