Commit Graph

5 Commits

Author SHA1 Message Date
Ian Wienand f76cfbab11 linters: fix spaces between filters
This fixes a number of places where we do not have spaces between
filters.  I think that this is a reasonable rule for readability (I
also think it probably was enforced, but maybe later versions got
better at detecting it?).

These are detected by a later version of Ansible lint; this change
should have no operational change to any roles but prepares us to
update in a follow-on change.

Change-Id: I07e1a109b87adce86f483d14d7e02fcecb8313d5
2022-07-27 17:13:39 +10:00
Tristan Cacqueray e2058a6b75 Fix deprecation warning of ansible-2.5
This change fixes this upcoming deprecation warning:

[DEPRECATION WARNING]: Using tests as filters is deprecated. Instead of using
`result|skipped` instead use `result is skipped`. This feature will be removed
in version 2.9.

Change-Id: Ic0c93f2ea896ead76922694e6150882d8a280daf
2018-06-04 00:17:18 +00:00
Monty Taylor 3bdb8d22bf Half-Revert "Revert "Add ensure-reno and ensure-babel roles""
Add the ensure-babel role back.

This reverts half of commit 89f8a9684c.

Change-Id: I1334ff1f469061884b222dd99e72a989d72c68be
2017-12-16 15:31:17 +01:00
Andreas Jaeger 89f8a9684c Revert "Add ensure-reno and ensure-babel roles"
This breaks building of releasenotes since we now have
both a version of this role in openstack-zuul-jobs and zuul-jobs. The other files expect that .venv/bin is used
but this change installs in user.

Revert for now - we can revert this again later once the full
stack is tested.

This reverts commit fa63c7b555.

Change-Id: I23ae1ed7abe7f36d46aae3d72293210ca2a8c0e6
2017-11-18 13:32:08 +00:00
Monty Taylor fa63c7b555
Add ensure-reno and ensure-babel roles
ensure-reno was written in openstack-zuul-jobs but it's not openstack
specific (or won't be once we remove the pip install . line)

ensure-babel is a similar role for installing babel.

Change-Id: I7827a63170168db5a6501b28d379e458171d556a
2017-11-17 09:04:08 -06:00