From: Anthony Liguori <anthony@codemonkey.ws>
To: Lucas Meneghel Rodrigues <lmr@redhat.com>
Cc: Scott Zawalski <scottz@google.com>,
Ademar Reis <areis@redhat.com>,
QEMU devel <qemu-devel@nongnu.org>,
Cleber Rosa <crosa@redhat.com>
Subject: Re: [Qemu-devel] [RFC] Future goals for autotest and virtualization tests
Date: Thu, 08 Mar 2012 08:48:33 -0600 [thread overview]
Message-ID: <4F58C6C1.5070700@codemonkey.ws> (raw)
In-Reply-To: <4F58BBA0.3070506@redhat.com>
On 03/08/2012 08:01 AM, Lucas Meneghel Rodrigues wrote:
> On 03/08/2012 10:36 AM, Anthony Liguori wrote:
>
>>> Virt/qemu tests: Minimal guest images
>>> -------------------------------------
>>>
>>> In order to make development level test possible, we need the tests to
>>> run fast.
>>> In order to do that, a set of minimal guest images is being developed
>>> and we
>>> have a version for x86_64 ready and functional:
>>>
>>> https://github.com/autotest/buildroot-autotest
>>
>> I'm really not a fan of buildroot. Note that in order to ship binaries,
>> full source needs to be provided in order to comply with the GPL. The
>> FSF at least states that referring to another website for source that's
>> not under your control doesn't satisfy the requirements of the GPL.
>
> We have a full clone of the buildroot repository that points out to the sources,
> if it's necessary to have a clone of all the projects needed host there in order
> to be able to publish a binary image to help people, then we can do it.
This is harder than I think you anticipate but okay..
>
>> Just out of curiosity, did you try to use qemu-test? Is there a reason
>> you created something different?
>
> I did, and it does what it proposes to. Nothing against it, but we have code
> that can do more things, that has been developed for longer time.
>
> It's similar to qemu-jeos vs buildbot, you have written scripts to create an
> image, which happens to be precisely why buildroot was written more than 10
> years ago and it works very well, allowing me to put things on the image that
> are not possible with qemu-jeos. If the problem is to point out to all sub
> modules as git repos, we can make that happen too, rather than re-writing stuff
> that works.
>
> For a long time I would like to see people working on a single code base,
I agree, we just disagree on what that code base should be :-)
That code base should be qemu.git. This discussion isn't about improving
third-party QE--at least not to me. Third party QE is a solved problem thanks
to all of your wonderful work with kvm-autotest. I'm sure you're looking for
more participation/developers, but even if you had twice the developers working
on kvm-autotest, I don't think it would fundamentally change our quality.
I'm interested in driving our development process toward test driven development
such that all 200+ people that write patches for QEMU for a given release write
and run tests as part of their normal development process. The requirements to
achieve this are different than the requirements you have been working against
up until now.
Every barrier that we put up to writing and running tests will reduce than
number of 200+ to something lower.
Submitting a patch to a different project than qemu.git is a barrier. Now
instead of getting a single set of feedback, you've got to deal with feedback
from two projects.
Having to use setup another framework (that runs as root) is another barrier. I
change a file in QEMU, run make, then run make check. I don't install anything,
I don't sudo anything. The whole process is relatively quick and painless.
Having to make a change to autotest, then install autotest, relaunch it, etc, is
just too complicated to be part of a developers fast path.
Now I think we should talk about how to make tests that live in qemu.git and run
as part of make check easily harnessed by autotest.. But I think the primary
focus of future test work needs to be within qemu.git.
Regards,
Anthony Liguori
> because that would allow things to progress further and people would have even
> better tools to use.
>
> By implementing the features of qemu-test in autotest we could simply use the
> qemu-test tests and use autotest rather than qemu test, and that's why we have
> done it.
>
next prev parent reply other threads:[~2012-03-08 14:49 UTC|newest]
Thread overview: 84+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-08 4:00 [Qemu-devel] [RFC] Future goals for autotest and virtualization tests Lucas Meneghel Rodrigues
2012-03-08 11:44 ` Stefan Hajnoczi
2012-03-08 11:54 ` Stefan Hajnoczi
2012-03-08 12:17 ` Ademar Reis
2012-03-08 12:18 ` Ademar Reis
2012-03-09 11:48 ` [Qemu-devel] [KVM-AUTOTEST] " Osier Yang
2012-03-08 12:28 ` [Qemu-devel] " Cleber Rosa
2012-03-08 13:06 ` Stefan Hajnoczi
2012-03-08 13:36 ` Anthony Liguori
2012-03-08 14:01 ` Lucas Meneghel Rodrigues
2012-03-08 14:48 ` Anthony Liguori [this message]
2012-03-08 15:00 ` Ademar Reis
2012-03-08 23:59 ` Andreas Färber
2012-03-09 0:08 ` Ademar Reis
2012-03-08 14:49 ` Ademar Reis
2012-03-08 14:56 ` Anthony Liguori
2012-03-08 15:07 ` Ademar Reis
2012-03-08 15:14 ` Anthony Liguori
2012-03-08 16:05 ` Ademar Reis
2012-03-08 17:03 ` Anthony Liguori
2012-03-08 17:59 ` Ademar Reis
2012-03-08 18:21 ` Lucas Meneghel Rodrigues
2012-03-08 18:22 ` Lucas Meneghel Rodrigues
2012-03-08 19:16 ` Anthony Liguori
2012-03-08 21:02 ` Ademar Reis
2012-03-08 21:24 ` Anthony Liguori
2012-03-08 22:24 ` Ademar Reis
2012-03-08 23:21 ` Anthony Liguori
2012-03-08 23:51 ` Ademar Reis
2012-03-09 9:41 ` Stefan Hajnoczi
2012-03-09 14:00 ` Ademar Reis
2012-03-09 14:54 ` Stefan Hajnoczi
2012-03-09 15:01 ` Ademar Reis
2012-03-09 15:17 ` Stefan Hajnoczi
2012-03-09 11:14 ` Kevin Wolf
2012-03-09 11:59 ` Anthony Liguori
2012-03-09 12:13 ` Kevin Wolf
2012-03-09 12:24 ` Anthony Liguori
2012-03-09 11:20 ` Cleber Rosa
2012-03-09 12:04 ` Anthony Liguori
2012-03-09 12:40 ` Cleber Rosa
2012-03-09 12:42 ` Anthony Liguori
2012-03-09 12:46 ` Cleber Rosa
2012-03-08 23:07 ` Lucas Meneghel Rodrigues
2012-03-08 23:56 ` Ademar Reis
2012-03-09 0:04 ` Anthony Liguori
2012-03-09 13:24 ` Paolo Bonzini
2012-03-09 12:13 ` Anthony Liguori
2012-03-09 12:48 ` Lucas Meneghel Rodrigues
2012-03-09 14:13 ` Anthony Liguori
2012-03-09 14:40 ` Lucas Meneghel Rodrigues
2012-03-09 14:40 ` Ademar Reis
2012-03-09 13:07 ` Paolo Bonzini
2012-03-09 13:56 ` Anthony Liguori
2012-03-09 14:43 ` Paolo Bonzini
2012-03-09 14:48 ` Anthony Liguori
2012-03-09 13:03 ` Paolo Bonzini
2012-03-08 15:46 ` Kevin Wolf
2012-03-08 15:57 ` Ademar Reis
2012-03-08 16:10 ` Anthony Liguori
2012-03-08 16:34 ` Kevin Wolf
2012-03-08 16:36 ` Anthony Liguori
2012-03-08 16:46 ` Ademar Reis
2012-03-08 16:47 ` Kevin Wolf
2012-03-08 16:08 ` Anthony Liguori
2012-03-08 15:19 ` Lucas Meneghel Rodrigues
2012-03-08 18:57 ` Anthony Liguori
2012-03-08 19:34 ` Lucas Meneghel Rodrigues
2012-03-08 19:43 ` Anthony Liguori
2012-03-08 20:17 ` Lucas Meneghel Rodrigues
2012-03-08 21:02 ` Andreas Färber
2012-03-08 21:03 ` Anthony Liguori
2012-03-09 13:36 ` Paolo Bonzini
2012-03-09 14:01 ` Anthony Liguori
2012-03-09 14:30 ` Paolo Bonzini
2012-03-09 14:43 ` Anthony Liguori
2012-03-09 15:00 ` Paolo Bonzini
2012-03-09 15:02 ` Anthony Liguori
2012-03-09 15:17 ` Paolo Bonzini
2012-03-09 15:24 ` Anthony Liguori
2012-03-09 15:34 ` Paolo Bonzini
2012-03-09 15:48 ` Anthony Liguori
2012-03-09 17:02 ` Cleber Rosa
2012-03-08 14:04 ` Alon Levy
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=4F58C6C1.5070700@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=areis@redhat.com \
--cc=crosa@redhat.com \
--cc=lmr@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=scottz@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;
as well as URLs for NNTP newsgroup(s).