[libvirt] [PATCH] admin: Fix the default uri for session daemon to libvirtd:///session

Martin Kletzander mkletzan at redhat.com
Mon Aug 1 09:27:45 UTC 2016


On Fri, Jul 29, 2016 at 03:40:28PM +0200, Michal Privoznik wrote:
>On 29.07.2016 14:08, Daniel P. Berrange wrote:
>> On Fri, Jul 29, 2016 at 02:06:00PM +0200, Erik Skultety wrote:
>>> Just like we decide on which URI we go with based on EUID for qemu in remote
>>> driver, do a similar thing for admin except we do not spawn a daemon in this
>>> case.
>>
>> IMHO we should auto-spawn the daemon too, so we have consistent
>> behaviour virsh virsh.
>>
>
>I don't think this is a good idea. The session daemon dies after 30
>seconds after last connection was closed. There's no point in setting
>something via admin API just to find it back on defaults after 1 minute
>because new daemon has been spawned. It's not like our domain APIs where
>libvirt refreshes the internal state of drivers upon their init.
>

Yeah, there's no point in auto-starting the daemon just to set
something.

>Michal
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160801/6e2e3944/attachment-0001.sig>


More information about the libvir-list mailing list