Re: [PATCH] virDomainSaveXML: Reject domains which name contain '/'

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

 



On 02/05/2013 08:54 AM, Michal Privoznik wrote:

>> I think you understate the benefits here. UUID based filenames are just
>> absolutely *ing horrific for an admin to see and work work and I am very
>> glad libvirt does not use such a naming scheme.
>>
>> Daniel
>>
> 
> But we discourage users to touch /etc/libvirt/qemu/*.xml files anyway.
> Moreover, we advise to define domain by not copying the files but using
> libvirt APIs.

It's not just /etc/libvirt/qemu/*.xml.  It's also
/var/log/libvirt/qemu/*.log.  And while we don't mind telling people to
leave /etc alone, we actively encourage people to use /var/log, and
anything we can do to make the logs more usable (names instead of uuids)
is worth it.

I'm in full agreement with this patch, but think it doesn't go far
enough; we probably also want to forbid domain names with a leading '.'.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]