[libvirt] [PATCH] Fix logging in libvirt_lxc controller

Cole Robinson crobinso at redhat.com
Tue Jun 16 18:44:28 UTC 2009


On 06/16/2009 01:35 PM, Amy Griffis wrote:
> The lxc controller can't see libvirtd's log level setting so it
> needs to re-query it from the environment. The parsing code has
> a few users now, so I added a new function to the internal API,
> virLogParseDefaultPriority() along the lines of the other parse
> functions.
> 

I'd say go the extra step and add something like virLogSetFromEnv, which
encapsulates the duplicate getenv calls as well.

There is also another duplicate of this code in tests/eventtest.c.

Thanks,
Cole




More information about the libvir-list mailing list