On 12/03/2012 09:06 AM, Martin Kletzander wrote: > Until now, virt-install supported only one seclabel and it was the > first one libvirt reported in capabilities. This patch adds support > for more of them and also adds a functionality to try to match the > right one from the label given. This is done by checking how many > colons the label has (precisely said, to how many parts it is split > by the colons). > --- > > I checked this without label as well as with selinux and DAC labels > and all worked as expected for me. However DAC labels in libvirt > 0.10.2 are supported only as uid:gid (e.g. 107:36), but that should be > handled in upper layers as we cannot know that the user/group exist on > remote machine. Newer versions of libvirt (>1.0.0) support user:group > (e.g. qemu:kvm) completely. > > v2: > - Changed according to Cole's suggestions > > virtinst/CapabilitiesParser.py | 10 +++++++--- > virtinst/Seclabel.py | 36 ++++++++++++++++++++++++++++++++---- > virtinst/VirtualDisk.py | 8 ++------ > 3 files changed, 41 insertions(+), 13 deletions(-) > This causes a few test failures when running 'python setup.py test', please make sure nothing there regresses. Feel free to ping me if you need help with any of the individual test cases. Thanks, Cole _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
