[Libguestfs] [nbdkit PATCH] error: Tweak filter name for easier debugging
Eric Blake
eblake at redhat.com
Fri Feb 4 13:47:49 UTC 2022
On Fri, Feb 04, 2022 at 09:32:54AM +0100, Laszlo Ersek wrote:
> On 02/03/22 22:29, Eric Blake wrote:
> > Grepping for the string 'error:' is complicated when the error filter
> > reports its name in the same manner. We can keep the filter .so and
> > file names the same, but tweak the filter .name field to change
> > messages using nbdkit_debug() to instead produce "error-inject:", so
> > that "error:" is reserved for uses of nbdkit_error().
> > ---
> >
> > I confused myself while trying to debug a new libnbd today, where
> > injecting errors was intentional. And then I noticed that 'man
> > nbdkit-error-filter' already tried to warn me. But we can do one step
> > better, as in this patch...
> >
>
> Very important patch, I'd say; log analysis is very common!
>
> Reviewed-by: Laszlo Ersek <lersek at redhat.com>
Thanks, now in as commit 886fd408 (and right after I pushed, I
remembered that I forgot to 'git am' my own patch to get the
Message-id: tag in the commit message. Oh well)
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
More information about the Libguestfs
mailing list