From: Michael Goldish <mgoldish@redhat.com>
To: Lucas Meneghel Rodrigues <lmr@redhat.com>
Cc: Autotest mailing list <autotest@test.kernel.org>,
KVM List <kvm@vger.kernel.org>, Martin Bligh <mbligh@google.com>
Subject: Re: [Autotest] Log message format in KVM-Autotest
Date: Mon, 8 Jun 2009 16:05:27 -0400 (EDT) [thread overview]
Message-ID: <1429044322.1504681244491527083.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com> (raw)
In-Reply-To: <16636675.1504651244491486554.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
----- "Lucas Meneghel Rodrigues" <lmr@redhat.com> wrote:
> On Mon, 2009-06-08 at 11:15 -0700, Martin Bligh wrote:
> > if it's specific to one test or whatever, you could also just put
> it inside the message?
No, because the test's 'shortname' contains both the test type and some
minimal information about the configuration in which it is run.
For example, scsi.smp2.Fedora.8.32.boot, is the KVM 'boot' test run with
Fedora.8.32, with 2 VCPUs and a scsi drive, and with the rest of the
parameters set to their most common values (e.g. the image format is qcow2
unless otherwise stated, the NIC is the default rtl8139...).
'boot' alone is not very informative.
> > Possibly with your own wrapper function around the logging?
>
> Yes, it's a possibility, but I'd rather avoid it. IMHO the current
> logging configuration is a good compromise between verbosity and the
> ability to debug problems on our code.
I would very much like to see the test name, the caller name and the time
in the logs. The KVM test is much more complex than other Autotest tests
and thus requires (in my opinion) more verbose logging. This seems very
reasonable to me. I personally find the logs produced by the new logger
after the merge very difficult to read, and even more difficult to debug.
Since the logging module seems to support most or all of what we need,
I see no reason not to use our own logging config file or our own Handler
and Formatter in the KVM test. I too would rather not resort to wrapper
functions.
> --
> Lucas Meneghel Rodrigues
> Software Engineer (QE)
> Red Hat - Emerging Technologies
next parent reply other threads:[~2009-06-08 20:05 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <16636675.1504651244491486554.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
2009-06-08 20:05 ` Michael Goldish [this message]
2009-06-08 10:35 Log message format in KVM-Autotest Michael Goldish
2009-06-08 13:03 ` Lucas Meneghel Rodrigues
2009-06-08 18:15 ` [Autotest] " Martin Bligh
2009-06-08 19:17 ` Lucas Meneghel Rodrigues
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1429044322.1504681244491527083.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com \
--to=mgoldish@redhat.com \
--cc=autotest@test.kernel.org \
--cc=kvm@vger.kernel.org \
--cc=lmr@redhat.com \
--cc=mbligh@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox