Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-11-10 Thread Andrea Bolognani
On Mon, 2020-11-09 at 11:37 -0500, Cleber Rosa wrote: > On Tue, Oct 20, 2020 at 07:52:43PM +0200, Andrea Bolognani wrote: > > On Sun, 2020-10-18 at 21:50 -0400, Cleber Rosa wrote: > > > +- name: Install basic packages to build QEMU on FreeBSD 12.x > > > + pkgng: > > > +# Originally

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-11-09 Thread Cleber Rosa
On Wed, Oct 21, 2020 at 09:00:27AM +0200, Thomas Huth wrote: > On 19/10/2020 03.50, Cleber Rosa wrote: > > To run basic jobs on custom runners, the environment needs to be > > properly set up. The most common requirement is having the right > > packages installed. > > > > The playbook introduced

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-11-09 Thread Cleber Rosa
On Tue, Oct 20, 2020 at 07:52:43PM +0200, Andrea Bolognani wrote: > On Sun, 2020-10-18 at 21:50 -0400, Cleber Rosa wrote: > > +++ b/scripts/ci/setup/build-environment.yml > > @@ -0,0 +1,233 @@ > > +--- > > +- name: Installation of basic packages to build QEMU > > + hosts: all > > + tasks: > > My

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-11-09 Thread Cleber Rosa
On Tue, Oct 20, 2020 at 08:18:54AM +0200, Erik Skultety wrote: > On Mon, Oct 19, 2020 at 04:25:31PM -0400, Cleber Rosa wrote: > > On Mon, Oct 19, 2020 at 12:27:41PM +0200, Erik Skultety wrote: > > > ... > > > > > > > diff --git a/scripts/ci/setup/inventory b/scripts/ci/setup/inventory > > > > new

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-10-21 Thread Thomas Huth
On 19/10/2020 03.50, Cleber Rosa wrote: > To run basic jobs on custom runners, the environment needs to be > properly set up. The most common requirement is having the right > packages installed. > > The playbook introduced here covers a number of different Linux > distributions and FreeBSD, and

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-10-20 Thread Andrea Bolognani
On Sun, 2020-10-18 at 21:50 -0400, Cleber Rosa wrote: > +++ b/scripts/ci/setup/build-environment.yml > @@ -0,0 +1,233 @@ > +--- > +- name: Installation of basic packages to build QEMU > + hosts: all > + tasks: My Ansible-fu is a bit rusty at the moment, so please double-check my claims and apolo

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-10-19 Thread Erik Skultety
On Mon, Oct 19, 2020 at 04:25:31PM -0400, Cleber Rosa wrote: > On Mon, Oct 19, 2020 at 12:27:41PM +0200, Erik Skultety wrote: > > ... > > > > > diff --git a/scripts/ci/setup/inventory b/scripts/ci/setup/inventory > > > new file mode 100644 > > > index 00..8bb7ba6b33 > > > --- /dev/null > >

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-10-19 Thread Cleber Rosa
On Mon, Oct 19, 2020 at 12:27:41PM +0200, Erik Skultety wrote: > ... > > > diff --git a/scripts/ci/setup/inventory b/scripts/ci/setup/inventory > > new file mode 100644 > > index 00..8bb7ba6b33 > > --- /dev/null > > +++ b/scripts/ci/setup/inventory > > @@ -0,0 +1,2 @@ > > +[local] > > Nit

Re: [PATCH v4 2/4] Jobs based on custom runners: build environment docs and playbook

2020-10-19 Thread Erik Skultety
... > diff --git a/scripts/ci/setup/inventory b/scripts/ci/setup/inventory > new file mode 100644 > index 00..8bb7ba6b33 > --- /dev/null > +++ b/scripts/ci/setup/inventory > @@ -0,0 +1,2 @@ > +[local] Nit pick, is a group for localhost actually needed? Regards, Erik