Re: [Qemu-devel] [PATCH 3/5] basevm: Call logging.basicConfig()

2017-09-28 Thread Lukáš Doktor
Dne 27.9.2017 v 15:03 Eduardo Habkost napsal(a): > Just setting level=DEBUG when debug is enabled is not enough: we > need to set up a log handler if we want debug messages generated > using logging.getLogger(...).debug() to be printed. > > This was not a problem before because logging.debug() cal

Re: [Qemu-devel] [PATCH 3/5] basevm: Call logging.basicConfig()

2017-09-27 Thread Eduardo Habkost
On Wed, Sep 27, 2017 at 09:35:44PM +0800, Fam Zheng wrote: > On Wed, 09/27 10:03, Eduardo Habkost wrote: > > Just setting level=DEBUG when debug is enabled is not enough: we > > need to set up a log handler if we want debug messages generated > > using logging.getLogger(...).debug() to be printed.

Re: [Qemu-devel] [PATCH 3/5] basevm: Call logging.basicConfig()

2017-09-27 Thread Fam Zheng
On Wed, 09/27 10:03, Eduardo Habkost wrote: > Just setting level=DEBUG when debug is enabled is not enough: we > need to set up a log handler if we want debug messages generated > using logging.getLogger(...).debug() to be printed. > > This was not a problem before because logging.debug() calls >

Re: [Qemu-devel] [PATCH 3/5] basevm: Call logging.basicConfig()

2017-09-27 Thread Daniel P. Berrange
On Wed, Sep 27, 2017 at 10:03:37AM -0300, Eduardo Habkost wrote: > Just setting level=DEBUG when debug is enabled is not enough: we > need to set up a log handler if we want debug messages generated > using logging.getLogger(...).debug() to be printed. > > This was not a problem before because log

[Qemu-devel] [PATCH 3/5] basevm: Call logging.basicConfig()

2017-09-27 Thread Eduardo Habkost
Just setting level=DEBUG when debug is enabled is not enough: we need to set up a log handler if we want debug messages generated using logging.getLogger(...).debug() to be printed. This was not a problem before because logging.debug() calls logging.basicConfig() implicitly, but it's safer to not