Fixed bugs identified by newer ansible-lint

- scm: git is mandatory on requirements.yml when using git urls
- role_name is needed in meta
- centos was not listed as supported platform
- galaxy_tags should be used instead of categories
- comparison to empty string should be avoided

These where found while attempting to enable functional testing using
ansible-molecule.

Change-Id: I5f8087516628e20c2539552c653d4ea5361234ec
Needed-By: https://review.opendev.org/#/c/675615/
This commit is contained in:
Sorin Sbarnea 2019-08-09 13:49:35 +01:00
parent 6f5e0518dd
commit 92b43f56db
3 changed files with 9 additions and 3 deletions

View File

@ -14,6 +14,7 @@
# limitations under the License. # limitations under the License.
galaxy_info: galaxy_info:
role_name: os_tempest
author: rcbops author: rcbops
description: Installation and setup of tempest description: Installation and setup of tempest
company: Rackspace company: Rackspace
@ -30,10 +31,13 @@ galaxy_info:
- name: EL - name: EL
versions: versions:
- 7 - 7
- name: CentOS
versions:
- 7
- name: opensuse - name: opensuse
versions: versions:
- 15 - 15
categories: galaxy_tags:
- cloud - cloud
- python - python
- tempest - tempest

View File

@ -1,8 +1,10 @@
# python_venv_build ansible role # python_venv_build ansible role
- name: python_venv_build - name: python_venv_build
scm: git
src: https://opendev.org/openstack/ansible-role-python_venv_build.git src: https://opendev.org/openstack/ansible-role-python_venv_build.git
version: master version: master
# ansible-config_template action plugin # ansible-config_template action plugin
- name: config_template - name: config_template
scm: git
src: https://opendev.org/openstack/ansible-config_template.git src: https://opendev.org/openstack/ansible-config_template.git
version: master version: master

View File

@ -151,7 +151,7 @@
copy: copy:
content: | content: |
{% for item in tempest_test_whitelist | unique | sort %} {% for item in tempest_test_whitelist | unique | sort %}
{% if item != '' %} {% if item %}
{{ item }} {{ item }}
{% endif %} {% endif %}
{% endfor %} {% endfor %}
@ -165,7 +165,7 @@
copy: copy:
content: | content: |
{% for item in tempest_test_blacklist %} {% for item in tempest_test_blacklist %}
{% if item.test != '' %} {% if item.test %}
{{ item.test }} {{ item.test }}
{% endif %} {% endif %}
{% endfor %} {% endfor %}