qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Niels de Vos <ndevos@redhat.com>
To: "Lukáš Doktor" <ldoktor@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
	areis@redhat.com, qemu-devel@nongnu.org,
	Xu Tian <xutian@redhat.com>, Hao Liu <hliu@redhat.com>
Subject: Re: [Qemu-devel] Automated testing of block/gluster.c with upstream Gluster
Date: Tue, 28 Jun 2016 17:56:23 +0200	[thread overview]
Message-ID: <20160628155623.GH10557@ndevos-x240.usersys.redhat.com> (raw)
In-Reply-To: <72a6e933-55fc-f448-a0bc-3861b030b73c@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 2813 bytes --]

On Tue, Jun 28, 2016 at 05:20:03PM +0200, Lukáš Doktor wrote:
> Dne 28.6.2016 v 16:10 Kevin Wolf napsal(a):
> > Am 28.06.2016 um 11:02 hat Niels de Vos geschrieben:
> > > Hi,
> > > 
> > > it seems we broke the block/gluster.c functionality with a recent patch
> > > in upstream Gluster. In order to prevent this from happening in the
> > > future, I would like to setup a Jenkins job that installs a plan CentOS
> > > with its version of QEMU, and nightly builds of upstream Gluster.
> > > Getting a notification about breakage the day after a patch got merged
> > > seems like a reasonable approach.
> > > 
> > > The test should at least boot the generic CentOS cloud image (slightly
> > > modified with libguestfs) and return a success/fail. I am wondering if
> > > there are automated tests like this already, and if I could (re)use some
> > > of the scripts for it. At the moment, I am thinking to so it like this:
> > >  - download the image [1]
> > >  - set kernel parameters to output on the serial console
> > >  - add a auto-login user/script
> > >  - have the script write "bootup complete" or something
> > >  - have the script poweroff the VM
> > >  - script that started the VM checks for the "bootup complete" message
> > >  - return success/fail
> > 
> > Sounds like something that Avocado should be able (or actually is
> > designed) to do. I can't tell you the details of how to write the test
> > case for it, but I'm adding a CC to Lukáš who probably can (and I think
> > it shouldn't be hard anyway).
> > 
> > Kevin
> > 
> 
> Hello guys,
> 
> yes, Avocado is designed to do this and I believe it even contain quite a
> few Gluster tests. You can look for them in avocado-vt or ping our QA folks
> who might give you some pointers (cc Xu nad Hao).
> 
> Regarding the building the CI I use the combination of Jenkins, Jenkins job
> builder and Avocado (avocado-vt) to check power/arm
> weekly/per-package-update. Jenkins even supports github and other triggers
> if you decide you have enough resources to check each PR/commit. It all
> depends on what HW you have available.

That looks promising! Its a bit more complex (or at least 'new' for me)
than that I was hoping. There is Gluster support in there, I found a
description of it here:
  http://avocado-vt.readthedocs.io/en/latest/GlusterFs.html
  http://avocado-vt.readthedocs.io/en/latest/RunQemuUnittests.html

Browsing through the docs does not really explain me how to put a
configuration file together that runs the QEMU tests with a VM image on
Gluster though. I probably need to read much more, but a pointer or very
minimal example would be much appreciated.

When I'm able to run avocado-vt, it should be trivial to put that in a
Jenkins job :)

Many thanks,
Niels

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2016-06-28 15:56 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-28  9:02 [Qemu-devel] Automated testing of block/gluster.c with upstream Gluster Niels de Vos
2016-06-28  9:41 ` Vasiliy Tolstov
2016-06-28 10:27   ` Niels de Vos
2016-06-28 10:54     ` Vasiliy Tolstov
2016-06-28 14:10 ` Kevin Wolf
2016-06-28 15:20   ` Lukáš Doktor
2016-06-28 15:56     ` Niels de Vos [this message]
2016-06-29  7:39       ` Lukáš Doktor
2016-06-29  9:55         ` Niels de Vos

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=20160628155623.GH10557@ndevos-x240.usersys.redhat.com \
    --to=ndevos@redhat.com \
    --cc=areis@redhat.com \
    --cc=hliu@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=ldoktor@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=xutian@redhat.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).