qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Lucas Meneghel Rodrigues <lmr@redhat.com>
Cc: 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 13:43:19 -0600	[thread overview]
Message-ID: <4F590BD7.6030605@codemonkey.ws> (raw)
In-Reply-To: <4F5909B3.4080405@redhat.com>

On 03/08/2012 01:34 PM, Lucas Meneghel Rodrigues wrote:
> On 03/08/2012 03:57 PM, Anthony Liguori wrote:
>> On 03/08/2012 09:19 AM, Lucas Meneghel Rodrigues wrote:
>>> Before I forget, I'd like to ask you about this:
>>>
>>> On 03/08/2012 10:36 AM, Anthony Liguori wrote:
>>>> 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.
>>>
>>> About using buildroot, what is up with it, since it is mature and
>>> works well?
>>> You mentioned than providing all the sources is harder than it looks
>>> like, and I
>>> surely think this might be the case.
>>
>> buildroot is a full blown distribution. But instead of distributing
>> binaries, it only distributes source code. Think of it like Gentoo--.
>>
>> It relies on third party links to fetch said source code which means
>> that it's not unusual
>
> By this definition, qemu test fetch source code from 3rd party repositories just
> as much, after all you have to fetch your linux and busybox code from somewhere,
> I assume.

It uses git submodules with repositories hosted on git.qemu.org.

>
> at all for buildroot to straight out fail. Because
>> it needs a GCC that can build shared libraries, it also requires
>> rebuilding GCC which increases the build time even for a minimalistic
>> configuration.
>
> But is this additional build time a problem? Is the person expected to be
> messing around with re-creating the minimal guest images all the time?

I think it's common enough to want to update the guest kernel that yes, this is 
something that many people need to be able to do in order for the test cases to 
be effective.  If it takes 24hr to rebuild the images, that's problematic.

>> Having a JeOS it not meant to replace having proper guests (like Fedora
>> or Ubuntu). As I've said in other notes, the point of having a JeOS is
>> to essentially use Linux as a libOS when writing unit tests.
>
> That is why we would have a reference binary jeos image that can be downloaded
> (although not a replacement, having a small guest makes setup time smaller, as
> well as smaller time to run the tests). If people need to tweak kernel, etc for
> some reason, I'm assuming that person would have their own clone of the repo and
> rebuild the images with the modification, and use that for the tests, I guess
> just the way you would do with qemu-jeos.

Yes, but this is a common thing.  This is something that needs to be designed 
for upfront for the infrastructure to be useful.

>>> But in all my naiveness, if the problem is to ship the exact source
>>> with the
>>> images have been built, couldn't I just ask buildroot to fetch all the
>>> tarball
>>> sources (there's a function to perform source download only) and add
>>> them to the
>>> appropriate git branch?
>>
>> Okay, let's say I'm a developer and I want to use a new Linux kernel to
>> test the new virtio-pci feature I added with buildroot. Where do I
>> begin? What patch do I submit?
>
> You change the configuration file to build your linux from git rather than the
> tarball, change the linux config and type 'make'. At the end, you'll have a
> patch that can be sent and kept in another autotest-buildroot branch, if it's
> proven to be useful *for other people*.

Except you also need to update those tarballs too that you're storing in git, 
remember.

Herein lies the problem.  You forgot and it's your proposal :-)

To make this model work, you're going to need to heavily change buildroot to use 
a local repository of tarballs and make sure that people update tarballs 
appropriately.  Updating tarballs is cumbersome and there's obviously a question 
of pedigree.  Are you going to require the tarballs come with the original 
release signature?  How are you going to guard against people taking tarballs of 
their own local trees instead of Linus' tree?

We already have this problem with the linux-headers FWIW.

qemu-jeos exists exactly to solve these problems in a robust fashion, based on 
the experiences we've had maintaining ROM modules in qemu.git.

Regards,

Anthony Liguori

>
> All in all, I don't see what is the big difference between this workflow and
> qemu-jeos.
>

  reply	other threads:[~2012-03-08 19:43 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
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 [this message]
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=4F590BD7.6030605@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=areis@redhat.com \
    --cc=crosa@redhat.com \
    --cc=lmr@redhat.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).