[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [virt-tools-list] [virt-install PATCH v2] Support multiple seclabels



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]