qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Ademar Reis <areis@redhat.com>
Cc: Lucas Meneghel Rodrigues <lmr@redhat.com>,
	Scott Zawalski <scottz@google.com>,
	QEMU devel <qemu-devel@nongnu.org>,
	"kvm-autotest@redhat.com" <kvm-autotest@redhat.com>,
	Cleber Rosa <crosa@redhat.com>
Subject: Re: [Qemu-devel] [RFC] Future goals for autotest and virtualization tests
Date: Thu, 08 Mar 2012 08:56:23 -0600	[thread overview]
Message-ID: <4F58C897.5020405@codemonkey.ws> (raw)
In-Reply-To: <20120308144958.GA25750@t420s.optimusnet>

On 03/08/2012 08:49 AM, Ademar Reis wrote:
> On Thu, Mar 08, 2012 at 07:36:11AM -0600, Anthony Liguori wrote:
>> On 03/07/2012 10:00 PM, Lucas Meneghel Rodrigues 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.
>>
>> Just out of curiosity, did you try to use qemu-test?  Is there a
>> reason you created something different?
>>
>> I think it's good that you're thinking about how to make writing
>> tests easier, but we have a growing test infrastructure in QEMU and
>> that's what I'd prefer people focused on.
>>
>
> You probably remember the long thread we had back in December on
> qemu-devel on this topic. Back then our message was "we have a
> growing test infrastructure in s/QEMU/autotest/ and that's what
> we'd prefer people focused on". :-)
>
>  From Dor:
>
> (http://lists.nongnu.org/archive/html/qemu-devel/2011-12/msg03024.html)
>
> """
> If you wish, you can challenge Lucas and Cleber w/ these type of
> requirements and we'll all improve as a result.
> """
>
> Your response was:
>
> """
> Well consider qemu-test the challenge. It's an existence proof
> that we can have a very easy to use framework for testing that
> runs extremely fast and is very easy to write tests for.
> """
>
> http://knowyourmeme.com/memes/challenge-accepted ;-)
>
> I particularly agreed with basically everything you said on that
> discussion regarding test simplification (I had just joined the
> team back then). To me, autotest has been focusing on QE-level,
> leaving the developer-level test requirements out. Now we're
> attacking this new front, and a lot of the requirements are
> indeed from that discussion.

If you want to talk about this in terms of "requirements", my requirement is for 
"developer-level" tests to live in qemu.git and be integrated into make check.

Just as we've been discussing and working on since the previous set of discussions.

> By simplifying the design and bringing barriers down, we hope to
> reach a broader audience and help developers write and maintain
> tests, benefiting from all the instrumentation that autotest
> brings. It's not going to be just about qemu (check the new test
> examples).
>
> We have a team fully dedicated to autotest and it's used not only
> by Qemu but also libvirt, Google, Xen, Fedora, Twitter, etc, etc
> (these all have code contributions in autotest)
>
> That said, the current qemu-tests will probably be easily
> integrated into (the new) autotest and we hope that, given enough
> time, autotest will be good enough to relieve qemu from the
> framework maintenance and code duplication with other projects.

autotest should not be the focal point for integration.  qemu.git should be.

I'd be perfectly happy to review patches submitting the test infrastructure from 
kvm-autotest into qemu.git (provided it didn't have unreasonable external 
dependencies and fit into QEMU).

Developer-level tests need to live where the developers live.  The developers 
live in qemu.git.  See my other response on this thread for the explanation of 
why this is so important.

Regards,

Anthony Liguori

  reply	other threads:[~2012-03-08 14:56 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
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 [this message]
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=4F58C897.5020405@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=areis@redhat.com \
    --cc=crosa@redhat.com \
    --cc=kvm-autotest@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).