qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@qumranet.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Suggestion for testing framework
Date: Thu, 05 Jun 2008 15:42:57 +0300	[thread overview]
Message-ID: <4847DF51.7050606@qumranet.com> (raw)
In-Reply-To: <18503.48210.408423.146974@mariner.uk.xensource.com>

Ian Jackson wrote:
> Paul Brook writes ("Re: [Qemu-devel] Suggestion for testing framework"):
>   
>> We were talking about a tester that does periodic long running tests off svn 
>> trunk, and reports the results. Individual developers are not directly 
>> involved.
>>     
>
> Just reporting the results is all very well, but as others have said
> testing is only useful with a very firm commitment to deal with the
> results of reports.  Few Free Software projects can manage to do this
> without some kind of formal and automatic linkage of QA passes into
> code distribution or propagation.  Without that, continuous discipline
> is needed - and if it ever slips, the test failures become an
> ever-deepening swamp.
>
> One common approach to this problem used with success in very
> different ways by both Debian and Xen and probably many others, is to
> maintain parallel branches: one (call it `unstable') is changed
> freely, and the other (call it `testing') is updated from unstable
> only when the QA criteria (whatever those are) are met.
>   

IMO parallel branches are overkill, especially given the current lack of 
reviewing and committing.

I'd suggest having just two rules:
- patch authors make a reasonable effort to ensure no regression in 
compilation or execution; this does not include running the entire test 
suite
- if a regression is discovered by the test suite, the offending patch 
is either fixed in a short time or reverted

This way progress can still be made, but quality is maintained

-- 
error compiling committee.c: too many arguments to function

  reply	other threads:[~2008-06-05 16:08 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-03 20:59 [Qemu-devel] Suggestion for testing framework Balazs Attila-Mihaly (Cd-MaN)
2008-06-03 21:50 ` Erik de Castro Lopo
2008-06-03 21:53   ` Anthony Liguori
2008-06-03 22:02   ` Paul Brook
2008-06-03 22:05     ` Glauber Costa
2008-06-03 22:17       ` Anthony Liguori
2008-06-03 22:25       ` Paul Brook
2008-06-03 22:35         ` Thiemo Seufer
2008-06-04  6:41           ` Laurent Desnogues
2008-06-04  7:49             ` Laurent Desnogues
2008-06-04  9:44             ` Edgar E. Iglesias
2008-06-04 10:28         ` Avi Kivity
2008-06-04 12:32           ` Glauber Costa
2008-06-04 12:36             ` Avi Kivity
2008-06-05 10:13         ` Ian Jackson
2008-06-05 12:42           ` Avi Kivity [this message]
2008-06-04 10:36 ` Fabrice Bellard
2008-06-15 18:52 ` [Qemu-devel][Patch] " Stefan Weil

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=4847DF51.7050606@qumranet.com \
    --to=avi@qumranet.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).