qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: Cleber Rosa <crosa@redhat.com>
Cc: qemu-devel@nongnu.org, berrange@redhat.com,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
	pbonzini@redhat.com, stefanha@redhat.com,
	"Peter Maydell" <peter.maydell@linaro.org>,
	eblake@redhat.com, "Kamil Rytarowski" <kamil@netbsd.org>
Subject: Re: [Qemu-devel] [PATCH v3 00/10] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)
Date: Fri, 25 Aug 2017 17:25:49 +0800	[thread overview]
Message-ID: <20170825092549.GH11465@lemon.lan> (raw)
In-Reply-To: <8805ba9c-a509-ed81-70e3-16b33bb1135c@redhat.com>

On Thu, 08/24 11:06, Cleber Rosa wrote:
> I replayed manually the FreeBSD VM setup, just to get a sense of how it
> could be automated.  Taking a few steps back, I realized that:
> 
>  * describing how to prepare a given OS to build QEMU is a generic task,
> not really bound to this VM setup
>  * cloud-init, while unarguably a sound solution, is bound to not only
> VMs, but to already prepared (with cloud-init support) images
> 
> What if we attempt to switch the "build environment setup" automation to
> use a more generic tool such as ansible?  Having an in-tree playbook
> that could be executed against a VM seems like slightly better than
> syncing the WIKI pages.
> 
> I'm no expert in ansible, but I can give it that a try if you fancy the
> idea.
> 

I have no objection if you want to give it a try. However it is basically a
separate problem, and I don't think this series would depend on that. The reason
is that setting up guest ssh is not part of the "build env setup" task, but is
the trickiest part here.  As you can see in the ubuntu image, "cloud-init" is
used just for that, and the build env is simply doing a "apt-get build-dep"; if
it were for Fedora the command is a dnf counterpart.  (I probably could have
removed the dev packages in BSD templates and add code to the *bsd scripts to
locally install packages after downloading.)

Ansible is probably an overkill anyway because what we'd have in the playbooks
are command lists to install package sets. Shell scripts can do that just fine,
this is not what ansible is great for, IMO (of course I'd be wrong if there is
an ansible module called 'build-dep' that supports many OSes, I'm not an ansible
expert either).

Fam

  reply	other threads:[~2017-08-25  9:26 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-22  4:41 [Qemu-devel] [PATCH v3 00/10] tests: Add VM based build tests (for non-x86_64 and/or non-Linux) Fam Zheng
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 01/10] gitignore: Ignore vm test images Fam Zheng
2017-08-22 11:07   ` Philippe Mathieu-Daudé
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 02/10] qemu.py: Add "wait()" method Fam Zheng
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 03/10] tests: Add vm test lib Fam Zheng
2017-08-22 11:19   ` Philippe Mathieu-Daudé
2017-08-22 12:33     ` Kamil Rytarowski
2017-08-22 14:00     ` Fam Zheng
2017-08-22 14:23       ` Philippe Mathieu-Daudé
2017-08-28 17:18         ` Fam Zheng
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 04/10] tests: Add ubuntu.i386 image Fam Zheng
2017-08-24 15:28   ` Cleber Rosa
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 05/10] tests: Add FreeBSD image Fam Zheng
2017-08-24 15:33   ` Cleber Rosa
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 06/10] tests: Add NetBSD image Fam Zheng
2017-08-23 15:15   ` Kamil Rytarowski
2017-08-24  9:25     ` Fam Zheng
2017-08-24 15:37   ` Cleber Rosa
2017-08-25  8:54     ` Fam Zheng
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 07/10] tests: Add OpenBSD image Fam Zheng
2017-08-24 15:34   ` Cleber Rosa
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 08/10] Makefile: Add rules to run vm tests Fam Zheng
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 09/10] MAINTAINERS: Add tests/vm entry Fam Zheng
2017-08-22  4:41 ` [Qemu-devel] [PATCH v3 10/10] tests: Add README for vm tests Fam Zheng
2017-08-22 13:38   ` Stefan Hajnoczi
2017-08-22 14:13     ` Fam Zheng
2017-08-22  7:53 ` [Qemu-devel] [PATCH v3 00/10] tests: Add VM based build tests (for non-x86_64 and/or non-Linux) no-reply
2017-08-22 13:45 ` Stefan Hajnoczi
2017-08-22 13:58 ` Paolo Bonzini
2017-08-22 14:23   ` Kamil Rytarowski
2017-08-22 14:29   ` Fam Zheng
2017-08-22 14:33 ` Kamil Rytarowski
2017-08-24  9:27   ` Fam Zheng
2017-08-24 15:06 ` Cleber Rosa
2017-08-25  9:25   ` Fam Zheng [this message]
2017-08-25 10:01     ` Paolo Bonzini

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=20170825092549.GH11465@lemon.lan \
    --to=famz@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=berrange@redhat.com \
    --cc=crosa@redhat.com \
    --cc=eblake@redhat.com \
    --cc=f4bug@amsat.org \
    --cc=kamil@netbsd.org \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@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).