Re: [libvirt] [PATCH] Updated to deal with specifying user IDs to that do not map to usernames
On Tue, Jun 07, 2016 at 08:24:14AM +0200, Peter Krempa wrote:
On Mon, Jun 06, 2016 at 14:25:23 -0500, Roy Keene wrote:
> Patch to libvirt master to avoid failing when a user ID is specified,
> e.g. for <seclabel type='dac'>, that does not map to a user name.
>
> This is useful if you want to run each VM as a separate user and not
> bother creating an /etc/passwd entry for each UID.
For this use case you shall prefix the name with a +. Please refer to
the documentation on seclabels.
Empirically that does not currently work:
# virsh dumpxml serial | grep --after 2 '<seclabel'
<seclabel type='static' model='dac' relabel='no'>
<label>+21421:+12421421</label>
</seclabel>
# virsh start serial
error: Failed to start domain serial
error: Cannot access storage file '/var/lib/libvirt/images/demo.qcow2' (as
uid:21421, gid:12421421): Success
# ls -al /var/lib/libvirt/images/demo.qcow2
-rw-r--r--. 1 21421 12421421 197120 Apr 30 2015 /var/lib/libvirt/images/demo.qcow2
Looking at the libvirtd logs we see
2016-06-07 14:49:13.724+0000: 13490: debug : qemuDomainCheckDiskPresence:3954 : Checking
for disk presence
2016-06-07 14:49:16.551+0000: 13490: error : virGetUserEnt:801 : Failed to find user
record for uid '21421'
2016-06-07 14:49:16.551+0000: 13490: error : virStorageFileGetMetadataRecurse:3114 :
Cannot access storage file '/var/lib/libvirt/images/demo.qcow2' (as uid:21421,
gid:12421421): Success
So even though the QEMU driver has honoured the '+' syntax, some of the
things QEMU is calling appears to be trying to resolve the UID back into
a user password record and failing.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|