Re: [PATCH 3/4] util: virlog: Introduce wildcard to log filters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 06, 2018 at 03:00:14PM +0200, Ján Tomko wrote:
> On Tue, Apr 03, 2018 at 10:45:48AM +0200, Erik Skultety wrote:
> > Since the introduction of log tuning capabilities to virt-admin by
> > @06b91785, this has been a much needed missing improvement on the way to
> > deprecate the global 'log_level'.
> >
> > Signed-off-by: Erik Skultety <eskultet@xxxxxxxxxx>
> > ---
> > src/remote/libvirtd.conf |  4 +++-
> > src/util/virlog.c        | 44 +++++++++++++++++++++++++++++++++++++++++++-
> > src/util/virlog.h        |  1 +
> > 3 files changed, 47 insertions(+), 2 deletions(-)
> >
>
> ACK
>
> > diff --git a/src/remote/libvirtd.conf b/src/remote/libvirtd.conf
> > index 9c0080dc0..7d823cf1a 100644
> > --- a/src/remote/libvirtd.conf
> > +++ b/src/remote/libvirtd.conf
> > @@ -368,7 +368,9 @@
> > # Multiple filters can be defined in a single @filters, they just need to be
> > # separated by spaces. Note that libvirt performs "first" match, i.e. if
> > # there are concurrent filters, the first one that matches will be applied,
> > -# given the order in log_filters.
> > +# given the order in log_filters with the exception of a wildcard filter, since
> > +# that's only taken into account if no other filter has matched, so
> > +# "4:* 1:util.pci" and "1:util.pci 4:*" are equivalent definitions.
> > #
>
> Also, "4:* 1:util.pci 1:*" is equivalent.

Sure, but if someone really tries that, 1) it doesn't make any sense 2) a
question on what the expected outcome they think should be is relevant,
therefore I'm not going to mention it.

Thanks,
Erik

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux