From: Dor Laor <dlaor@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Glauber Costa <glommer@redhat.com>,
Paul Brook <paul@codesourcery.com>,
qemu-devel@nongnu.org
Subject: autotest (was Re: [Qemu-devel] Re: [Qemu-commits] [COMMIT f80f9ec] Convert machine registration to use module initfunctions)
Date: Sat, 23 May 2009 17:22:07 +0300 [thread overview]
Message-ID: <4A18068F.5020700@redhat.com> (raw)
In-Reply-To: <4A160138.7050109@us.ibm.com>
Anthony Liguori wrote:
> Glauber Costa wrote:
>> How about we start sending patches to the list? Then this kind of
>> thing can be avoided.
>>
>
> It doesn't magically solve the problem. I post most patches and still
> regressions slip in. I review every patch I commit and still
> regressions slip in. People are imperfect.
>
> The best way to prevent regressions is to have an automated test suite
> that everyone can use to validate that a series of patches doesn't
> break things.
We have such a frame work, it's the kvm autotest project -
http://www.linux-kvm.org/page/KVM-Autotest.
It is used currently by Avi for his tree + to test more internal trees.
It recently catched a regression with -net usage.
Don't let its name fool you, it should be able to test bare qemu code
too. Also we are finally on track for
merging the changes upstream towards autotest project.
So ideally, what we need for qemu project is:
1. Make kvm-autotest work with qemu only
- It's mainly about changing timeouts and better, move to kickstart
files
2. Have qemu community use it and contribute to it.
3. Maintainers should ...
- Well, I'll let you maintainers complete that line :)
#1 will surely happen soon. I hope #2, #3 will join right after, making
qemu head/stable a better place.
Regards,
Dor
next prev parent reply other threads:[~2009-05-23 14:23 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200905211348.n4LDmnYd025976@d01av04.pok.ibm.com>
2009-05-21 22:36 ` [Qemu-devel] Re: [Qemu-commits] [COMMIT f80f9ec] Convert machine registration to use module initfunctions Paul Brook
2009-05-21 22:50 ` Glauber Costa
2009-05-22 1:34 ` Anthony Liguori
2009-05-22 1:49 ` Glauber Costa
2009-05-22 2:00 ` Anthony Liguori
2009-05-23 14:22 ` Dor Laor [this message]
2009-05-24 19:46 ` autotest (was Re: [Qemu-devel] Re: [Qemu-commits] [COMMIT f80f9ec] Convert machine registration to use module initfunctions) Anthony Liguori
2009-05-24 20:50 ` Avi Kivity
2009-05-26 8:10 ` Anthony Liguori
2009-05-26 9:17 ` Avi Kivity
2009-05-26 9:17 ` Dor Laor
2009-05-26 9:30 ` Anthony Liguori
2009-05-26 9:54 ` Dor Laor
2009-05-22 1:21 ` [Qemu-devel] Re: [Qemu-commits] [COMMIT f80f9ec] Convert machine registration to use module initfunctions Anthony Liguori
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=4A18068F.5020700@redhat.com \
--to=dlaor@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=glommer@redhat.com \
--cc=paul@codesourcery.com \
--cc=qemu-devel@nongnu.org \
/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;
as well as URLs for NNTP newsgroup(s).