Commit Graph

7 Commits

Author SHA1 Message Date
Guillaume Chauvel 21169062e6 prepare-workspace: Add Role Variable in README.rst
Change-Id: I9bd85cbbf334a8d8c57b4adabf633e7046c273fd
2020-06-22 21:17:38 +02:00
Albin Vass cb9ef441ca prepare-workspace: do not synchronize owner
Change-Id: I173657611f2ebafe3efd92dad381535824d94264
2020-05-18 17:05:32 +02:00
Paul Belanger 5e31d28504 Delete files in dest that don't exist
When using static nodes, we don't properly delete stale files nolonger
tracked in git.

Change-Id: Icb3b140abd1e2e3f1ca127f91f78c0df7efc4298
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2019-04-01 09:18:26 -04:00
Monty Taylor 5cd8fbeb58
Update ANSIBLE_LIBRARY to use envsitepackagesdir
We install zuul via test-requirements, so the zuul files should be in a
directory under the site-packges of the virtualenv that tox installed it
in to. Update the path to point correctly to that location.

Remove the ansible-lint skip tags which should now work because the
library path should be pointing to a location that actually holds the
content.

Change-Id: If2d4b39267c4b9a3102a951143b568f8447af8d9
2018-10-18 09:23:25 -05:00
Colleen Murphy d1956bd9f7 Fix ansible-lint job
The 'linters' tox environment was not running on roles, it was only
running on playbooks. This change adds a command to the linters
environment to ensure all roles are linted.

Since these weren't being linted, there were some problems with them.
The first was a warning about usage of the shell module. Both of these
usages seemed appropriate, so this patch adds a skip_ansible_lint tag to
each task that was failing. The second is a warning "no action detected
in task" for zuul modules. This is due to the fact that the linter
cannot find the custom module. One option is to set the ANSIBLE_LIBRARY
path to point to the zuul ansible library directory[1], but the linter
virtualenv does not actually have zuul installed. Instead, we just
disable the linter for the failing tasks.

This also cleans up a comment in the tox file that was referring to a
nonexistent zuul job.

[1] https://github.com/willthames/ansible-review/issues/16

Change-Id: Ie49da9a09733b623bb25c5a4c8aa07eacacf4b33
2018-03-28 23:10:22 +02:00
Jamie Lennox 0ae663192b Allow overriding the workspace directory in prepare-workspace
The prepare-workspace role currently assumes that we are going to copy
our code into the home directory of the user. This may not always be the
case and so allow overriding this variable.

Change-Id: I6d640015dc764770f8a468edbcf7ec2ea3421e20
2017-09-04 17:25:30 +10:00
James E. Blair cfffd4431b Move some base job roles from openstack-zuul-roles
Change-Id: I7ac8d2335518c06808461f2fb30ea896709f09ed
2017-07-13 15:13:41 -07:00