[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PULL 1/2] block/gluster: add support to c
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PULL 1/2] block/gluster: add support to choose libgfapi logfile |
Date: |
Tue, 13 Sep 2016 10:23:32 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
On 09/13/2016 12:42 AM, Jeff Cody wrote:
> From: Prasanna Kumar Kalever <address@hidden>
>
> currently all the libgfapi logs defaults to '/dev/stderr' as it was hardcoded
> in a call to glfs logging api. When the debug level is chosen to DEBUG/TRACE,
> gfapi logs will be huge and fill/overflow the console view.
>
> This patch provides a commandline option to mention log file path which helps
> in logging to the specified file and also help in persisting the gfapi logs.
>
> +++ b/qapi/block-core.json
> @@ -2154,13 +2154,16 @@
> #
> # @debug-level: #optional libgfapi log level (default '4' which is Error)
> #
> +# @logfile: #optional libgfapi log file (default /dev/stderr)
> +#
> # Since: 2.7
logfile missed 2.7; please add a '(since 2.8)' designator on the
@logfile line as a followup patch.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature