[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 3/4] Jobs based on custom runners: docs and gitlab-runner
From: |
Cleber Rosa |
Subject: |
Re: [PATCH v6 3/4] Jobs based on custom runners: docs and gitlab-runner setup playbook |
Date: |
Tue, 29 Jun 2021 20:18:30 -0400 |
On Thu, Jun 10, 2021 at 2:24 AM Thomas Huth <thuth@redhat.com> wrote:
>
> On 08/06/2021 05.14, Cleber Rosa wrote:
> > To have the jobs dispatched to custom runners, gitlab-runner must
> > be installed, active as a service and properly configured. The
> > variables file and playbook introduced here should help with those
> > steps.
> >
> > The playbook introduced here covers the Linux distributions and
> > has been primarily tested on OS/machines that the QEMU project
> > has available to act as runners, namely:
> >
> > * Ubuntu 20.04 on aarch64
> > * Ubuntu 18.04 on s390x
> >
> > But, it should work on all other Linux distributions. Earlier
> > versions were tested on FreeBSD too, so chances of success are
> > high.
> >
> > Signed-off-by: Cleber Rosa <crosa@redhat.com>
> > ---
> > docs/devel/ci.rst | 57 ++++++++++++++++++++++++++++
> > scripts/ci/setup/.gitignore | 1 +
> > scripts/ci/setup/gitlab-runner.yml | 61 ++++++++++++++++++++++++++++++
> > scripts/ci/setup/vars.yml.template | 12 ++++++
> > 4 files changed, 131 insertions(+)
> > create mode 100644 scripts/ci/setup/.gitignore
> > create mode 100644 scripts/ci/setup/gitlab-runner.yml
> > create mode 100644 scripts/ci/setup/vars.yml.template
> [...]
> > diff --git a/scripts/ci/setup/.gitignore b/scripts/ci/setup/.gitignore
> > new file mode 100644
> > index 0000000000..f112d05dd0
> > --- /dev/null
> > +++ b/scripts/ci/setup/.gitignore
> > @@ -0,0 +1 @@
> > +vars.yml
> > \ No newline at end of file
>
> Add a newline, please.
>
Sure! Thanks for spotting that.
> > diff --git a/scripts/ci/setup/gitlab-runner.yml
> > b/scripts/ci/setup/gitlab-runner.yml
> > new file mode 100644
> > index 0000000000..98dab92bb5
> > --- /dev/null
> > +++ b/scripts/ci/setup/gitlab-runner.yml
> > @@ -0,0 +1,61 @@
> > +---
> > +- name: Installation of gitlab-runner
> > + hosts: all
> > + vars_files:
> > + - vars.yml
> > + tasks:
> > + - debug:
> > + msg: 'Checking for a valid GitLab registration token'
> > + failed_when: "gitlab_runner_registration_token ==
> > 'PLEASE_PROVIDE_A_VALID_TOKEN'"
>
> Could you please add a comment at the top of the file or name it differently
> so that it is clear from a quick glance that this is an ansible playbook?
> Poeple might later wonder otherwise...
>
Good point. I'm adding a proper header with copyright / comment.
> Thomas
>
Thanks for the review.
- Cleber.